-
Notifications
You must be signed in to change notification settings - Fork 20
Contributing
RossLO edited this page Apr 28, 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
- Check all of the workflows of the latest merged PR are passed here
- Review the release note which created by the release drafter
- Click the "Publish release" button below the editing release note to trigger
Publish
workflow - Ensure the
Publish
workflow is done here - Check the latest npm version of glints-aries is updated here
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
- Run
yarn storybook
to check that the icon appears as expected locally- It runs
build:icon
which creates the icon file using the script in generateIcons.ts
- It runs
- Follow the How to Contribute section