mirror of https://github.com/portainer/portainer
78 lines
3.0 KiB
Markdown
78 lines
3.0 KiB
Markdown
# Contributing Guidelines
|
|
|
|
Some basic conventions for contributing to this project.
|
|
|
|
### General
|
|
|
|
Please make sure that there aren't existing pull requests attempting to address the issue mentioned. Likewise, please check for issues related to update, as someone else may be working on the issue in a branch or fork.
|
|
|
|
* Please open a discussion in a new issue / existing issue to talk about the changes you'd like to bring
|
|
* Develop in a topic branch, not master/develop
|
|
|
|
When creating a new branch, prefix it with the *type* of the change (see section **Commit Message Format** below), the associated opened issue number, a dash and some text describing the issue (using dash as a separator).
|
|
|
|
For example, if you work on a bugfix for the issue #361, you could name the branch `fix361-template-selection`.
|
|
|
|
### Issues open to contribution
|
|
|
|
Want to contribute but don't know where to start?
|
|
|
|
Some of the open issues are labeled with prefix `exp/`, this is used to mark them as available for contributors to work on. All of these have an attributed difficulty level:
|
|
|
|
* **beginner**: a task that should be accessible with users not familiar with the codebase
|
|
* **intermediate**: a task that require some understanding of the project codebase or some experience in
|
|
either AngularJS or Golang
|
|
|
|
You can have a use Github filters to list these issues:
|
|
|
|
* beginner labeled issues: https://github.com/portainer/portainer/labels/exp%2Fbeginner
|
|
* intermediate labeled issues: https://github.com/portainer/portainer/labels/exp%2Fintermediate
|
|
|
|
### Linting
|
|
|
|
Please check your code using `grunt lint` before submitting your pull requests.
|
|
|
|
### Commit Message Format
|
|
|
|
Each commit message should include a **type**, a **scope** and a **subject**:
|
|
|
|
```
|
|
<type>(<scope>): <subject>
|
|
```
|
|
|
|
Lines should not exceed 100 characters. This allows the message to be easier to read on github as well as in various git tools and produces a nice, neat commit log ie:
|
|
|
|
```
|
|
#271 feat(containers): add exposed ports in the containers view
|
|
#270 fix(templates): fix a display issue in the templates view
|
|
#269 style(dashboard): update dashboard with new layout
|
|
```
|
|
|
|
#### Type
|
|
|
|
Must be one of the following:
|
|
|
|
* **feat**: A new feature
|
|
* **fix**: A bug fix
|
|
* **docs**: Documentation only changes
|
|
* **style**: Changes that do not affect the meaning of the code (white-space, formatting, missing
|
|
semi-colons, etc)
|
|
* **refactor**: A code change that neither fixes a bug or adds a feature
|
|
* **test**: Adding missing tests
|
|
* **chore**: Changes to the build process or auxiliary tools and libraries such as documentation
|
|
generation
|
|
|
|
#### Scope
|
|
|
|
The scope could be anything specifying place of the commit change. For example `networks`,
|
|
`containers`, `images` etc...
|
|
You can use the **area** label tag associated on the issue here (for `area/containers` use `containers` as a scope...)
|
|
|
|
#### Subject
|
|
|
|
The subject contains succinct description of the change:
|
|
|
|
* use the imperative, present tense: "change" not "changed" nor "changes"
|
|
* don't capitalize first letter
|
|
* no dot (.) at the end
|