Contributions are very much welcome!
- Fork this repo
- Create a namespaced branch from the
main
branch, based on what you'd like to work on. e.g.,fix/submit-btn
,feat/add-img2img
, etc.
feat/...
orfeature/...
: (new feature for the user, not a new feature for build script)fix/...
: (bug fix for the user, not a fix to a build script)docs/...
: (changes to the documentation)style/...
: (formatting, missing semi colons, etc; no production code change)refactor/...
: (refactoring production code, eg. renaming a variable)test/...
: (adding missing tests, refactoring tests; no production code change)chore/...
: (updating webpack tasks etc; no production code change)
- Make commits to your feature branch, preferably using conventional commits:
feat: added a new feature
fix: fix inconsistent tests [fixes #0]
refactor: ...
chore: ...
test: ...
doc: ...
- See further examples here
- When you've finished with your fix or feature, rebase upstream changes into your branch and then submit a pull request directly to
main
. Include a description of your changes. - Your pull request will be reviewed by another maintainer. The point of code reviews is to help keep the codebase clean and of high quality and, equally as important, to help you grow as a programmer. If your code reviewer requests you make a change you don't understand, ask them why.
- Fix any issues raised by your code reviewer, and push your fixes as a new commit.
- Once the pull request has been reviewed, it will be merged by another member of the team. Do not merge your own commits.
- Uphold the current code standard:
- Keep your code DRY.
- Apply the boy scout rule.
- Run any tests (if applicable) before submitting a pull request.
This is just to help you organize your process
- Did I cut my work branch off of
main
? - Did I follow the correct naming convention for my branch?
- Is my branch focused on a single main change?
- Do all of my changes directly relate to this change?
- Did I rebase the upstream
main
branch after I finished all my work? - Did I write a clear pull request message detailing what changes I made?
- Did I get a code review?
- Did I make any requested changes from that code review?
If you follow all of these guidelines and make good changes, you should have no problem getting your changes merged in.