Skip to content

CodeOwners pattern for default reviewers of all PullRequests#1536

Merged
jsed-nhs merged 2 commits intomainfrom
APPT-XXX/code-owners
Apr 17, 2026
Merged

CodeOwners pattern for default reviewers of all PullRequests#1536
jsed-nhs merged 2 commits intomainfrom
APPT-XXX/code-owners

Conversation

@jsed-nhs
Copy link
Copy Markdown
Contributor

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Fixes # (issue)

Checklist:

  • My work is behind a feature toggle (if appropriate)
  • If my work is behind a feature toggle, I've added a full suite of tests for both the ON and OFF state
  • The ticket number is in the Pull Request title, with format "APPT-XXX: My Title Here"
  • I have ran npm tsc / lint (in the future these will be ran automatically)
  • My code generates no new .NET warnings (in the future these will be treated as errors)
  • If I've added a new Function, it is disabled in all but one of the terraform groups (e.g. http_functions)
  • If I've added a new Function, it has both unit and integration tests. Any request body validators have unit tests also
  • If I've made UI changes, I've added appropriate Playwright and Jest tests
  • If I've added/updated an end-point, I've added the appropriate annotations and tested the Swagger documentation reflects the change

pata9
pata9 previously approved these changes Mar 20, 2026
@sonarqubecloud
Copy link
Copy Markdown

@jsed-nhs jsed-nhs merged commit 4597eeb into main Apr 17, 2026
5 checks passed
@jsed-nhs jsed-nhs deleted the APPT-XXX/code-owners branch April 17, 2026 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants