UNPKG

7.71 kBMarkdownView Raw
1# Contributing
2
3I'm really glad you're reading this, because we need volunteer developers to help this project come to fruition.
4
5We love to receive contributions from our community — you! There are many ways to contribute, from writing tutorials or blog posts, improving the documentation, submitting bug reports and feature requests or writing code which can be incorporated into project itself.
6
7Following these guidelines helps to communicate that you respect the time of the developers managing and developing this open source project. In return, they should reciprocate that respect in addressing your issue, assessing changes, and helping you finalize your pull requests.
8
9These are mostly guidelines, not rules. Use your best judgment, and feel free to propose changes to this document in a pull request.
10
11
12## Code of Conduct
13
14This project has adopted a Code of Conduct that we expect project participants to adhere to. Please read [the full text](https://github.com/lykmapipo/permission/blob/master/CODE_OF_CONDUCT.md) so that you can understand what actions will and will not be tolerated.
15
16
17## Open Development
18
19All work happens directly on [GitHub](https://github.com/lykmapipo/permission). Both core team members and external contributors send pull requests which go through the same review process.
20
21
22## Branch Organization
23
24We will do our best to keep the [`master` branch](https://github.com/lykmapipo/permission/tree/master) in good shape, with tests passing at all times.
25
26If you send a pull request, please do it against the [`master` branch](https://github.com/lykmapipo/permission/tree/master).
27
28
29## Semantic Versioning
30
31This project follows [semantic versioning](http://semver.org/). We release patch versions for bugfixes, minor versions for new features, and major versions for any breaking changes.
32
33
34## Where to Find Known Issues
35
36We are using [GitHub Issues](https://github.com/lykmapipo/permission/issues) for all issues. Before filing a new task, try to make sure your problem doesn't already exist.
37
38
39## Proposing a Change
40
41If you intend to change the public API, or make any non-trivial changes to the implementation, we recommend [filing an issue](https://github.com/lykmapipo/permission/issues/new). This lets us reach an agreement on your proposal before you put significant effort into it.
42
43If you're only fixing a bug, it's fine to submit a pull request right away but we still recommend to file an issue detailing what you're fixing. This is helpful in case we don't accept that specific fix but want to keep track of the issue.
44
45
46## Your First Pull Request
47
48Working on your first Pull Request? You can learn how from this free video series:
49
50**[How to Contribute to an Open Source Project on GitHub](https://egghead.io/series/how-to-contribute-to-an-open-source-project-on-github)**
51
52If you decide to fix an issue, please be sure to check the comment thread in case somebody is already working on a fix. If nobody is working on it at the moment, please leave a comment stating that you intend to work on it so other people don't accidentally duplicate your effort.
53
54If somebody claims an issue but doesn't follow up for more than two weeks, it's fine to take it over but you should still leave a comment.
55
56
57## Sending a Pull Request
58
59The core team is monitoring for pull requests. We will review your pull request and either merge it, request changes to it, or close it with an explanation. We'll do our best to provide updates and feedback throughout the process.
60
61
62## Style Guide
63
64Look at [Airbnb's Style Guide](https://github.com/airbnb/javascript) will guide you in the right direction.
65
66## Git Commit Guidelines
67
68We have very precise rules over how our git commit messages can be formatted. This leads to **more readable messages** that are easy to follow when looking through the **project history**. But also, we use the git commit messages to **generate change log**.
69
70The commit message formatting can be added using a typical git workflow or through the use of a CLI wizard ([Commitizen](https://github.com/commitizen/cz-cli)). To use the wizard, run `npm run cmt` in your terminal after staging your changes in git.
71
72### Commit Message Format
73Each commit message consists of a **header**, a **body** and a **footer**. The header has a special format that includes a **type**, a **scope** and a **subject**:
74
75```
76<type>(<scope>): <subject>
77<BLANK LINE>
78<body>
79<BLANK LINE>
80<footer>
81```
82
83The **header** is mandatory and the **scope** of the header is optional.
84
85Any line of the commit message cannot be longer 100 characters!. This allows the message to be easier to read on GitHub as well as in various git tools.
86
87### Revert
88If the commit reverts a previous commit, it should begin with `revert: `, followed by the header of the reverted commit. In the body it should say: `This reverts commit <hash>.`, where the hash is the SHA of the commit being reverted. A commit with this format is automatically created by the [`git revert`](https://git-scm.com/docs/git-revert) command.
89
90### Type
91Must be one of the following:
92
93* **feat**: A new feature
94* **fix**: A bug fix
95* **docs**: Documentation only changes
96* **style**: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
97* **refactor**: A code change that neither fixes a bug nor adds a feature
98* **perf**: A code change that improves performance
99* **test**: Adding missing or correcting existing tests
100* **chore**: Changes to the build process or auxiliary tools and libraries such as documentation generation
101
102### Scope
103The scope could be anything specifying place of the commit change. You can use `*` when the change affects more than a single scope.
104
105### Subject
106The subject contains succinct description of the change:
107
108* use the imperative, present tense: "change" not "changed" nor "changes"
109* don't capitalize first letter
110* no dot (.) at the end
111
112### Body
113Just as in the **subject**, use the imperative, present tense: "change" not "changed" nor "changes". The body should include the motivation for the change and contrast this with previous behavior.
114
115### Footer
116The footer should contain any information about **Breaking Changes** and is also the place to [reference GitHub issues that this commit closes](https://help.github.com/articles/closing-issues-using-keywords/).
117
118**Breaking Changes** should start with the word `BREAKING CHANGE:` with a space or two newlines. The rest of the commit message is then used for this.
119
120
121## License
122
123By contributing, you agree that your contributions will be licensed under its MIT license.
124
125
126## Developer's Certificate of Origin 1.1
127
128By making a contribution to this project, I certify that:
129
130* (a) The contribution was created in whole or in part by me and I
131 have the right to submit it under the open source license
132 indicated in the file; or
133
134* (b) The contribution is based upon previous work that, to the best
135 of my knowledge, is covered under an appropriate open source
136 license and I have the right under that license to submit that
137 work with modifications, whether created in whole or in part
138 by me, under the same open source license (unless I am
139 permitted to submit under a different license), as indicated
140 in the file; or
141
142* (c) The contribution was provided directly to me by some other
143 person who certified (a), (b) or (c) and I have not modified
144 it.
145
146* (d) I understand and agree that this project and the contribution
147 are public and that a record of the contribution (including all
148 personal information I submit with it, including my sign-off) is
149 maintained indefinitely and may be redistributed consistent with
150 this project or the open source license(s) involved.