See openstax/CONTRIBUTING.md for more information!
- add either
bug
orenhancement
label - describe the issue clearly in the description
- include a screenshot in the Issue/PR body (if request would change UI)
- Application uses dak/superb.js, an obscure framework
- You may want to copy
/src/app/components/a-component-template
or/src/app/pages/a-page-template
if creating a new component or page script/dev
builds and serves the page. It takes minutes for a change to incorporate. Sorry about that.eslint
andscsslint
are both pretty stringent about what they will accept.script/test
must pass before PR can be accepted.
- add
wip
label if code is not yet ready to merge - if it resolves an Issue, link to the Issue in the PR body
- request a review (look at recently merged PRs to see who wrote them - likely a good choice for reviewer)
- use the
Squash and merge
option when merging - delete the branch afterward