- Code should follow Microsoft's C# code conventions
- Classes should use file-scoped namespaces and be in the format
NHS.CohortManager.<Tests>.Service
i.eNHS.CohortManager.CohortDistributionService - Test names should follow the convention
NameOfMethod_Scenario_ExpectedResult
- Commit names should follow the conventional commits standard
i.erefactor: move class out of shared - Branch names should follow the following format:
type/DTOSS-<ticket-number>-<pr-description>
i.efix/DTOSS-1234-fixed-a-bug - Pull request names should follow the same conventional commits standard as commits
- Write a bullet point description of the changes made in your PR
- Add a link to the ticket in the context section as well as any other context
you think would be helpful for reviewers to know - PR comments are to be marked resolved by the reviewer, not the author
- Write the function & tests
- Add XML docs
- Create a Dockerfile and add to the relevant compose file
- Add a health check
- Update Terraform (or ask the platform team to do it)