Skip to content

Commit 54b9b4e

Browse files
authored
222 contributing guidelines (#230)
Closes #228
1 parent c44a7f0 commit 54b9b4e

File tree

2 files changed

+26
-0
lines changed

2 files changed

+26
-0
lines changed

.github/CONTRIBUTING.md

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
# Welcome to the Engineering Framework contributors guide
2+
Thank you for your interest in improving the framework.
3+
4+
In this guide you will get an overview of the contribution workflow from opening an issue, creating a PR, reviewing, and merging the PR.
5+
6+
The framework is concerned with engineering best practice at NHSD, however, operational matters (e.g. structure of teams, roles and responsibilities, etc.) are out of scope. Please consider this when adding new content, and if in doubt, any of the code owners will be more than happy to discuss.
7+
8+
If you're new to Github and/or Markdown, Github's own [contributor's guide](https://github.com/github/docs/blob/main/CONTRIBUTING.md) provides good signposting on these topics.
9+
10+
## Using issues
11+
If you have a change (or a new page) to raise, please create an [issue](https://github.com/NHSDigital/software-engineering-quality-framework/issues) first, and reference it in your commits. Before raising a new issue, have a look through the existing ones in case it's already been raised (Framework owners will strive to keep the list small enough).
12+
13+
## Pull requests and merging
14+
You can't push to the main branch. Therefore, for all changes you will need to create a new branch, and then a pull request to merge said branch into main. The framework owners will be nominated as reviewers of your PR by default, but feel free to add other people as well if you think they will have valuable input.
15+
16+
When committing your changes, please reference the issue they're resolving, so it will be closed automatically (e.g. `Closes #123`).
17+
18+
## Contributing guidelines and etiquette
19+
* Preview your Markdown code to make sure the format is not broken.
20+
* Check grammar, spelling and punctuation, no one wants to look pedantic by requesting changes due to typos or inconsistent grammar/syntax, but it's only fair to keep this tidy.
21+
* The framework is open to the world. This has a few implications:
22+
* Nothing in it should be confidential, private to NHSD or include any personal data.
23+
* All links in framework pages should be public.

README.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,3 +35,6 @@ The framework consists of:
3535
* [Product Development Cloud PaaS Working Group](communities/pd-cloud-working-group.md)
3636
* [Secure Engineering CoP](communities/security-cop.md)
3737
* Guidance on how to practice [continuous improvement](continuous-improvement.md)
38+
39+
## Contributing
40+
See our [contributor's guide](./.github/CONTRIBUTING.md)

0 commit comments

Comments
 (0)