We love pull requests from everyone. By participating in this project, you agree to abide by the thoughtbot code of conduct. Here’s a quick guide:
While developing and contributing to Bitters you can use Parcel to
spin up an example site to help preview your changes! Run npm run contrib
to pull in the stylesheets and serve a small HTML preview page
at http://localhost:1234
.
-
Fork the repository.
-
Make your changes in a topic branch.
-
Make sure you have Node.js & npm installed:
$ npm -v
If you don't have Node.js or npm, you can download it here.
-
Next up,
cd
in to Bitters and runnpm install
:$ cd bitters $ npm install
Then run the Parcel build command:
$ npm run contrib
This will have Parcel compile the stylesheets from
core/
and link them to a test page so you can preview your changes! -
Squash your commits into a single one (more on that here).
-
Rebase against
origin/master
, push to your fork and submit a pull request.
At this point you're waiting on us. We like to at least comment on, if not accept, pull requests within three business days (and, typically, one business day). We may suggest some changes or improvements or alternatives.
Some things that will increase the chance that your pull request is accepted:
- Fix a bug, refactor code or expand an existing feature.
- Use the right syntax and naming conventions.
- Update parts of the documentation that are affected by your contribution.
- Capitalize your commit messages.
- Start your message with a verb.
- Use present tense.
- Refer to the issue/PR number in your squashed commit message.
- Two spaces, no tabs.
- Dashes instead of underscores or camel case:
section-header
notsection_header
orsectionHeader
- Names should be descriptive and written in full-words:
$font-color--base
not$color
or$txtCLR
- Space between property and value:
width: 20px
notwidth:20px
- Declarations within a block should be ordered alphabetically.
- Blank lines between rules.
- No trailing whitespace. Blank lines should not have any space.
For more on how thoughtbot writes Sass, check out our guides.