You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
* CCM-11207: documentation overview and structure for developer and consumer guidance
* CCM-11207: more pages & placeholders
* CCM-11207: ensure all deps
* CCM-11207: change deploy to use jekyll site
* CCM-11207: ensure directories present
* CCM-11207: need a spec to gen from
* CCM-11207: rename generate-includes to avoid impacting other CI
* CCM-11207: missed rename
* CCM-11207: remove the repos entry
* CCM-11207: review comments: spelling and repo changes
* CCM-11207: fix title
You are not permitted to push directly to the remote `main` branch in GitHub.
22
+
23
+
Changes to `main` when require approval(s) recorded on your PR.
24
+
Required approvers are controlled by the CODEOWNERS file but in summary:
25
+
26
+
- Infrastructure changes should be reviewed by DevOps team members
27
+
- Workflow/action changes should be reviewed by Maintainers of the repo
28
+
- All other changes should be review by NHS API Development team members
29
+
30
+
Merges should only take place:
31
+
32
+
- They are intended for the next release cycle
33
+
- All CI workflows have completed successfully
34
+
35
+
## Coding Standards
36
+
37
+
Your PR must follow all agreed coding standards for the project. Terraform and CI/CD standards are listed in sections below.
38
+
39
+
### GitHooks
40
+
41
+
GitHooks are available within this repo to help maintain standards and protect against e.g. secrets disclosure
42
+
43
+
GitHooks **must** be configured and run on commits before pushing to remote. Refer to the developer documentation for more information if required.
44
+
45
+
## Testing Your Branch
46
+
47
+
You can test your branch in a dynamic environment prior to merging to `main`. These are created as part of the `cicd-1-pull-request.yaml` workflow, triggered when a PR is created or updated.
0 commit comments