-
Notifications
You must be signed in to change notification settings - Fork 20
Contributing
Francis Goh edited this page Oct 26, 2019
·
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:
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}"
Run tests locally before pushing code up. We are in the midst of implementing continuous integration and deployment into this project.
For more information on testing
yarn upgrade glints-aries
For glints engineers, you can run this command on your review apps.
- Add your icon to the IconLibrary file
- Remove all unnecessary html attributes from the
<path>
(e.g.id
)
- Remove all unnecessary html attributes from the
- Follow the How to Contribute section