Skip to content
This repository was archived by the owner on Mar 26, 2024. It is now read-only.

Commit e82ac3b

Browse files
jpettobassrock
andauthored
docs(open source): add contributing and code of conduct docs (#326)
* docs(open source): add contributing and code of conduct docs * Update CONTRIBUTING.md Co-authored-by: Daniel Brooks <[email protected]> Co-authored-by: Daniel Brooks <[email protected]>
1 parent b1255b7 commit e82ac3b

File tree

2 files changed

+113
-0
lines changed

2 files changed

+113
-0
lines changed

CODE_OF_CONDUCT.md

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
# Community Participation Guidelines
2+
3+
This repository is governed by Mozilla's code of conduct and etiquette guidelines.
4+
For more details, please read the
5+
[Mozilla Community Participation Guidelines](https://www.mozilla.org/about/governance/policies/participation/).
6+
7+
## How to Report
8+
9+
For more information on how to report violations of the Community Participation Guidelines, please read our '[How to Report](https://www.mozilla.org/about/governance/policies/participation/reporting/)' page.

CONTRIBUTING.md

Lines changed: 104 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,104 @@
1+
## Contributing
2+
3+
Thank you for checking out our Collection API work. We welcome contributions from everyone! By participating in this project, you agree to abide by the Mozilla [Community Participation Guidelines](https://www.mozilla.org/about/governance/policies/participation/).
4+
5+
### Asking questions / receiving updates
6+
7+
- Slack channel (Pocket staff only): #p-curation-tools
8+
9+
- File issues/questions on Github: [https://github.com/Pocket/collection-api/issues](https://github.com/Pocket/collection-api/issues). We typically triage new issues every Monday.
10+
11+
### Finding Bugs & Filing Tickets
12+
13+
If you've found a bug, or have a feature idea that you you'd like to see, follow these simple guidelines:
14+
15+
- Pick a thoughtful and concise title for the issue (ie. _not_ Thing Doesn't Work!)
16+
17+
- Make sure to mention your local configuration, OS and basic system parameters
18+
19+
- If you can reproduce the bug, give a step-by-step recipe
20+
21+
- Include stack traces from the console(s) where appropriate
22+
23+
- Screenshots and screen recordings welcome!
24+
25+
- When in doubt, take a look at some existing issues and emulate
26+
27+
### Contributing Code
28+
29+
If you are new to the repo, you might want to pay close attention to these tags, as they are typically a great way to get started: Good First Bug, Bug, Chore, and Polish. If you see a bug that is not yet assigned to anyone, start a conversation with an engineer in the ticket itself, expressing your interest in taking the bug. If you take the bug, someone will set the ticket to Assigned to Contributor, so we can be proactive about helping you succeed in fixing the bug.
30+
31+
When you have some code written, you can open up a Pull Request, get your code reviewed, and see your code merged into the codebase.
32+
33+
### Setting up your development environment
34+
35+
Please review the [README](https://github.com/Pocket/collection-api/blob/main/README.md) for instructions on setting up your development environment, installing dependencies and building the extensions.
36+
37+
### Creating Pull Requests
38+
39+
You have identified the bug, written code and now want to get it into the main repo using a [Pull Request](https://help.github.com/articles/about-pull-requests/).
40+
41+
All code is added using a pull request against the master branch of our repo. Before submitting a PR, please go through this checklist:
42+
43+
- All tests must pass (and if you haven't written a test, please do!)
44+
45+
- Fill out the pull request template as outlined
46+
47+
- Please add a PR / Needs Review tag to your PR (if you have permission). This starts the code review process. If you cannot add a tag, don't worry, we will add it during triage.
48+
49+
- Make sure your PR will merge gracefully with master at the time you create the PR, and that your commit history is 'clean'
50+
51+
### Understanding Code Reviews
52+
53+
You have created a PR and submitted it to the repo, and now are waiting patiently for you code review feedback. One of the projects module owners will be along and will either:
54+
55+
- Make suggestions for some improvements
56+
57+
- Give you a 👍 in the comments section, indicating the review is done and the code can be merged
58+
59+
Typically, you will iterate on the PR, making changes and pushing your changes to new commits on the PR. When the reviewer is satisfied that your code is good-to-go, you will get the coveted R+ comment, and your code can be merged. If you have commit permission, you can go ahead and merge the code to master, otherwise, it will be done for you.
60+
61+
Our project prides itself on its respectful, patient and positive attitude when it comes to reviewing contributors' code, and as such, we expect contributors to be respectful, patient and positive in their communications as well.
62+
63+
### Commit format
64+
65+
Commits should be formated as `type(scope): message`
66+
67+
The following types are allowed:
68+
69+
| Type | Description |
70+
| -------- | ----------------------------------------------------------------------------------------------------- |
71+
| feat | A new feature |
72+
| fix | A bug fix |
73+
| docs | Documentation only changes |
74+
| style | Changes that do not affect the meaning of the code (white-space, formatting,missing semi-colons, etc) |
75+
| refactor | A code change that neither fixes a bug nor adds a feature |
76+
| perf | A code change that improves performance |
77+
| test | Adding missing or correcting existing tests |
78+
| chore | Changes to the build process or auxiliary tools and libraries such as documentation generation |
79+
80+
### Writing Good Git Commit Messages
81+
82+
We use standard changelog and enforce [conventional commits](https://www.conventionalcommits.org/).
83+
84+
We like this overview by Chris Beams on "[How to Write a Git Commit Message](https://chris.beams.io/posts/git-commit/)".
85+
86+
The tl;dr is:
87+
88+
1. [Separate subject from body with a blank line](https://chris.beams.io/posts/git-commit/#separate)
89+
90+
2. [Limit the subject line to 50 characters](https://chris.beams.io/posts/git-commit/#limit-50)
91+
92+
3. [Capitalize the subject line](https://chris.beams.io/posts/git-commit/#capitalize)
93+
94+
4. [Do not end the subject line with a period](https://chris.beams.io/posts/git-commit/#end)
95+
96+
5. [Use a verb to start your subject line (Add, Remove, Fix, Update, Rework, Polish, etc.)](https://chris.beams.io/posts/git-commit/#imperative)
97+
98+
6. [Wrap the body at 72 characters](https://chris.beams.io/posts/git-commit/#wrap-72)
99+
100+
7. [Use the body to explain _what_ and _why_ vs. _how_](https://chris.beams.io/posts/git-commit/#why-not-how)
101+
102+
### Understanding How Pocket Triages
103+
104+
The project team meets weekly (in a closed meeting, for the time being), to discuss project priorities, to triage new tickets, and to redistribute the work amongst team members. Any contributors tickets or PRs are carefully considered, prioritized, and if needed, assigned a reviewer.

0 commit comments

Comments
 (0)