-
Notifications
You must be signed in to change notification settings - Fork 61
Commit Guide
Here goes rules about commits. The instructions on how to write commits for this project follows the specifications in Conventional Commits and is based on the Angular Convention.
The key words “MUST”, “MUST NOT”, “REQUIRED”, “SHALL”, “SHALL NOT”, “SHOULD”, “SHOULD NOT”, “RECOMMENDED”, “MAY”, and “OPTIONAL” in this document are to be interpreted as described in RFC 2119.
The commit message should be structured as follows:
[optional emoji]<type>[optional scope]: <description>
[optional body]
[optional footer(s)]
An emoji can be used to give more context to the commit message.
Some examples can be found here.
-
feat- Changes that includes a new feature (functionality) in the code base. -
fix- Changes that solves a problem (bug) in the code base. -
refactor- Changes that does not add functionality or remove bugs, but performs some code improvement. -
test- Changes that add or correct tests in the code base. -
docs- Changes that modifies only the code documentation. -
build- Change that modifies how the code should be executed. -
ci- Changes to our CI configuration files and scripts. -
style- Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc).- If 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.
- If the commit reverts a previous commit, it should begin with
A scope MAY be provided to a commit’s type, to provide additional contextual information and is contained within parenthesis, e.g., feat(parser): add ability to parse arrays.
A description MUST immediately follow the colon and space after the type/scope prefix. The description is a short summary of the code changes, e.g., fix: resolve the array parsing issue when multiple spaces were contained in string.
- use the imperative, present tense: "change" not "changed" nor "changes"
- don't capitalize the first letter
- no dot (.) at the end
A longer commit body MAY be provided after the short description, providing additional contextual information about the code changes. The body MUST begin one blank line after the description.
A commit body is free-form and MAY consist of any number of newline separated paragraphs.
One or more footers MAY be provided one blank line after the body. Each footer MUST consist of a word token, followed by either a :<space> or <space># separator, followed by a string value (this is inspired by the git trailer convention).
A footer’s token MUST use - in place of whitespace characters, e.g., Acked-by (this helps differentiate the footer section from a multi-paragraph body). An exception is made for BREAKING CHANGE, which MAY also be used as a token.
A footer’s value MAY contain spaces and newlines, and parsing MUST terminate when the next valid footer token/separator pair is observed.
Breaking changes MUST be indicated in the type/scope prefix of a commit, or as an entry in the footer.
If included as a footer, a breaking change MUST consist of the uppercase text BREAKING CHANGE, followed by a colon, space, and description, e.g., BREAKING CHANGE: environment variables now take precedence over config files.
If included in the type/scope prefix, breaking changes MUST be indicated by a ! immediately before the :. If ! is used, BREAKING CHANGE: MAY be omitted from the footer section, and the commit description SHALL be used to describe the breaking change.
Ref to Conventional Commits Examples.
feat: allow provided config object to extend other configs
BREAKING CHANGE: `extends` key in config file is now used for extending other config files
feat!: send an email to the customer when a product is shipped
feat(api)!: send an email to the customer when a product is shipped
chore!: drop support for Node 6
BREAKING CHANGE: use JavaScript features not available in Node 6.
docs: correct spelling of CHANGELOG
feat(lang): add Polish language
fix: prevent racing of requests
Introduce a request id and a reference to latest request. Dismiss
incoming responses other than from latest request.
Remove timeouts which were used to mitigate the racing issue but are
obsolete now.
Reviewed-by: Z
Refs: #123
Made based on Microsoft wiki for vscode.
Project Management
Contributing
- How to Contribute
- Source Code Organization
- Coding Guidelines
- Working on Guides
Documentation