Thank you for your interest in contributing to the AeroGear project. We want keep this process as easy as possible so we've outlined a few guidelines below. For more information about the AeroGear community and project , visit our website.
Whether you're contributing a new feature or bug fix, or simply submitting a ticket, the AeroGear team is available for technical advice or feedback. You can reach us at #unified-push on Discord or join the mailing list -- both are actively monitored.
- Make sure you have a JIRA account
- Make sure you have a GitHub account
- Submit a ticket for your issue to the
AeroGear project, assuming one does
not already exist.
- Clearly describe the issue including steps to reproduce when it is a bug.
- Make sure you fill in the earliest version that you know has the issue.
- Fork the repository on GitHub.
- Create a topic branch from where you want to base your work.
- This is usually the master branch.
- To quickly create a topic branch based on master;
git checkout -b <branch name> master
. By convention we typically include the JIRA issue key in the branch name, e.g.AEROGEAR-1234-my-feature
. - Please avoid working directly on the
master
branch.
- Make commits of logical units.
- Ensure your commit messages comples with conventional commits rules, e.g. "chore: fix spelling mistake in README."
- Follow the coding style in use.
- Run
npm run fix
before committing. - Make sure you have added the necessary tests for your changes.
- Run all the tests to assure nothing else was accidentally broken.
- Push your changes to a topic branch in your fork of the repository.
- Submit a pull request to the repository in the [AeroGear GitHub organization]
(https://github.com/aerogear) and choose branch you want to patch
(usually master).
- Advanced users may want to install the GitHub CLI
and use the
hub pull-request
command.
- Advanced users may want to install the GitHub CLI
and use the
- Update your JIRA ticket to mark that you have submitted code and are ready
for it to be reviewed (Status: Dev Complete).
- Include a link to the pull request in the ticket.
- Add detail about the change to the pull request including screenshots if the change affects the UI.
- After submitting a pull request, one of AeroGear team members will review it.
- Changes may be requested to conform to our style guide and internal requirements.
- When the changes are approved and all tests are passing, a AeroGear team member will merge them.
- Note: if you have write access to the repository, do not directly merge pull requests. Let another team member review your pull request and approve it.