-
Notifications
You must be signed in to change notification settings - Fork 19
Contributing
Ting Lee edited this page Jun 29, 2020
·
18 revisions
- Clone the
glints-aries
repo- For Glints engineers: you do not have to fork the repo. This allows other engineers to easily access your branches.
- Create a branch (see Branch-Naming)
- Commit your changes and push the code
- Submit a PR
- request a review from the Glints Aries maintainer (currently @franciscrispin)
- assign yourself to the PR
- add appropriate label(s)
- reference any relevant issues or PRs by pasting the links in the comments
- resolve any merge conflicts before submitting the PR
- Once the PR is approved, the reviewer will merge the PR and publish a new version of the library
- New features or enhancements:
feature/feature-name
- Bugs or hotfixes:
hotfix/hotfix-name
- Writing tests:
test/component-name
- Documentation:
doc/documentation-name
We are in the process of documenting how we define test coverage that is "good".
Example:
yarn test
to run tests locally. We are in the midst of implementing continuous integration and deployment into this project.
If you are using vscode, download the prettier extension and turn on Editor: Format On Save
Run this command to format all subdirectories and files in a directory:
yarn prettier --write "*src/directoryName/**/*.{tsx,ts}"
For more information on testing
yarn upgrade glints-aries@version
For glints engineers, you can run this command on your review apps.