-
Notifications
You must be signed in to change notification settings - Fork 22
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #255 from BuildingSync/docs/developer-resources
docs: add notes on git naming conventions
- Loading branch information
Showing
1 changed file
with
46 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,46 @@ | ||
# Developer Resources | ||
|
||
## Git Naming Conventions | ||
|
||
Commit messages should follow the format of | ||
|
||
|
||
```bash | ||
<type>[( optional scope )]: <description> | ||
|
||
[optional body] | ||
``` | ||
|
||
`type` must be one of the following: | ||
|
||
- **docs**: Changes to the documentation (e.g. improving the annotation of an element, updating this file, etc) | ||
- **feat**: Adds a new feature (e.g. adding a new element to the schema) | ||
- **fix**: A bug fix (e.g. adding a missing closing tag, or moving a misplaced element) | ||
- **proposal**: Adding or editing a proposal (e.g. creating a proposal for a new modeling concept for the schema) | ||
- **refactor**: Changes that don't fix a bug or add a new feature (e.g. turning an element into a complexType for reuse) | ||
- **style**: Changes that don't affect the meaning of code (e.g. whitespace) | ||
- **test**: Adding or correcting tests | ||
|
||
`scope` is optional for commit messages, and should indicate the general area of the application affected. | ||
|
||
`description` is a short description of the changes in imperative present tense (such as “add function to _”, not “added function”) | ||
|
||
Branches should be named as `[optional issue number -]<type>/<scope>`, where `scope` is the general scope affected, or if creating a feature branch, a shortened name of the feature being added. If `scope` is more than one word, it should be separated by dashes. | ||
|
||
Pull Request titles should follow the format `[# optional issue number] <type>[( optional scope )]: <description>`, following the same conventions as commit messages. | ||
|
||
Commit examples: | ||
|
||
- `feat(schema): add MyElement to ParentElement` | ||
- `proposal: add proposal for MyElement` | ||
- `refactor: make SimpleElement restriction a regex test` | ||
|
||
Branch examples: | ||
|
||
- `1234-feat/typical-operating-hours` | ||
- `refactor/terminal-unit-type` | ||
|
||
Pull request examples: | ||
|
||
- `#1234 feat(schema): add MyElement to ParentElement` | ||
- `proposal: add proposal for MyElement` |