👍🎉 First off, thanks for taking the time to contribute! We appreciate you! 🎉👍
The following is a set of guidelines for contributing to the AllClear Web App.
In the main README you will find a section on Running the Project
Help review open PR's. If you're comfortable reviewing the PR's, please do so.
-
Pull down the PR branch locally and test the work.
git checkout -t origin/[branch_name] -
If everything works well, and it’s a low risk PR into dev, go ahead and approve the PR and merge it via Github. If not, leave comments and request changes.
- Grab a ticket from the top of the backlog. See Default (Unassigned) tickets here
- Off of the
devbranch, create a new branch with the ticket number e.g.
git checkout dev
git pull
git checkout -b AC-152- Do the work When your work is complete, merge dev back into your branch, push up your local branch and create a pull request on Github to merge the branch into dev branch. e.g.
# add your changes
git add someFile.js
# commit with the ticket number
git commit -m "AC-152 fix: header misalignment"
# merge any changes to dev that were made while you were working on your branch
git pull origin dev
# run linting locally, to ensure your PR passes the lint checks . Fix any issues
npm run lint
# push your local branch back up to remote
git push origin AC-152- Create a PR on github
- Once you've created your PR, CI tests will automatically run directly on the PR. Once these are done (in a few minutes), you should see "All checks have passed" . If you don't, fix the issues.
Thank you again! We look forward to seeing your code in the dev branch soon 👩💻 👨💻 🚤