Skip to content
This repository has been archived by the owner on Feb 7, 2025. It is now read-only.

Update DoD tasks in Story template #758

Merged
merged 1 commit into from
Jan 9, 2024
Merged

Update DoD tasks in Story template #758

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 1 addition & 3 deletions .github/ISSUE_TEMPLATE/story.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,11 +34,9 @@ As a _, so that _, I need _.
- [ ] Main `README.md`
- [ ] Other READMEs in the repo
- [ ] If applicable, update the `ReportStream Setup` section in `README.md`
- [ ] Threat model updated
- [ ] [Threat model](https://lucid.app/lucidchart/8c6e8d37-2612-42a8-ac57-150c83f8e29e/edit) updated
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can anyone with access to this link edit it?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think so

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Correct, that isn't a public link. It works if you are logged in, but not if you aren't logged in.

- [ ] API documentation updated
- [ ] Source code documentation created when the code is not self-documenting
- [ ] Code quality tasks completed
- [ ] Logging added where useful
- [ ] Code refactored for clarity and no design/technical debt
- [ ] Adhere to separation of concerns; code is not tightly coupled, especially to 3rd party dependencies
- [ ] Code is reviewed or developed by pair; 1 approval is needed but consider requiring an outside-the-pair reviewer
Expand Down
Loading