-
Notifications
You must be signed in to change notification settings - Fork 1
Work Agreement
Manuel Cepeda edited this page Sep 10, 2020
·
1 revision
All documentation must be written in Spanish. On the other hand, all code must be written in English.
- Gradual review (3 days) for the issues.
Must match with the following conventions:
-
It must start with the main purpose of the change,
-
feature
: new functionality -
fix
: solves an error -
setup
: installation and/or configuration
-
-
It must end with the issue number,
- Examples,
feature/issue-90
orfix/issue-91
- Examples,
- First of all, all branches must use the branch
development
as a starting point. - To merge your branch into
development
, a pull request must be opened with the following:- Title, it could have the issue number.
- Description, must be a detailed description of the changes not only the commit messages.
- 2 Reviewers, to perform code reviews.
- Optional, you can open the pull request as a draft for yourself and perform a code review of your code before publishing it.
- To merge
development
intostage
,- 3 Reviewers must approve change.
- To merge
stage
intomain
,- Any maintainer can perform this change at this point.
- pre-commit: prettier & eslint
- pre-push: jest
- post-push: gatsby build