Skip to content

Latest commit

 

History

History
148 lines (104 loc) · 4.7 KB

CONTRIBUTING.md

File metadata and controls

148 lines (104 loc) · 4.7 KB

Contributing

Overview

  • Make your changes.
  • Update NEWS.md.in.
  • Update .mailmap if necessary.
  • Write a test covering your feature or fix.
  • Ensure existing and new tests are passing.
  • Submit a pull request on GitHub.

Explanation

Consider updating NEWS.md.in. The topmost section is for the upcoming release. Bugfixes should be marked with BUGFIX. Small things (typos, code style) should be grouped but with multiple authors (Documentation updates thanks to Dan Croak and Roberto Pedroso).

We use your name and email address as produced by git-shortlog(1). You can change how this is formatted by modifying .mailmap. More details on that file can be found in the git Documentation/mailmap.txt.

It is mandatory to include tests with pull requests. You must ensure that the existing test suite passes with any changes you make. Also, any attempts to add or extend tests will increase the chances of your pull request being merged.

Submit a pull request using GitHub. If there is a relevant bug, mention it in the commit message (Fixes #42.). We love pull requests from everyone. By participating in this project, you agree to abide by the thoughtbot code of conduct.

Setup

  1. Fork the repo.
  2. Install dependencies
  • Cram is used for tests: pip install cram
  • The mustache gem for building the HTML pages: gem install mustache
  1. Prepare the build system: ./autogen.sh. (This depends on GNU autoconf and GNU automake.)
  2. Configure the package: ./configure.
  3. Make sure the tests pass: make check.
  4. Start hacking

Testing

The test suite uses cram. It is an integration suite, meaning the programs are exercised from the outside and assertions are made only on their output or effects.

The test suite requires Perl with the Cwd module. It expects to find Perl as perl in $PATH.

All tests can be run like so:

$ make check

Individual tests can be run like so:

$ env TESTS=test/lsrc-dotfiles-dirs.t make -e check

If you intend to write a new test:

  1. Add your test at test/subcommand-something-meaningful.t.
  2. Add the relative name to the TESTS variable in Makefile.am.
  3. Source test/helper.sh as the first line of your test.
  4. When in doubt, use existing tests as a guide.

Governance

Your interaction with this project can be divided into three sections: you as a contributor, you as a committer, and how to become a committer.

Commenter and meta-contributor

Those commenting and triaging issues and pull reports are expected to adhere to our code of conduct.

Contributor

This is a slow-moving project. The maintainers' goal is to provide a yearly release.

As a contributor, you can expect a maintainer to add a GitHub label to your pull request or issue within two weeks. This indicates that a maintainer has seen your contribution and quickly triaged it. If a maintainer believe that your feature request will not be merged, they will tell you as much during the triage step.

Bug reports, either as issues or as pull requests, get the maintainers' priority. The maintainers consider documentation bugs to be as important as code bugs.

Feature contributions (that is, a pull request that add a new feature) get lower priority. A maintainer will evaluate the patch carefully for maintainability and to ensure the test coverage is high.

Feature requests without a patch will be closed. A maintainer will try to provide a description of how to write your patch while we close the request, though they cannot guarantee this.

Interactions outside of GitHub Issues and Pull Requests are not considered contributing. For example, tweeting about a bug in rcm is not contributing to rcm, and therefore we can make no guarantees or promises about such an action.

You are expected to adhere to our code of conduct.

Committer

As a committer, you can merge at any time. The maintainers encourage you to have the code reviewed by someone beforehand. Those in the thoughtbot team can ping the @thoughtbot/shell group.

You are expected to adhere to our code of conduct.

Maintainer

A maintainer acts as a project manager and, as such, has final say and responsibility.

All maintainers must adhere to our code of conduct.

Promotion

All current thoughtbot employees are committers to rcm.

Contributors with two merged patches are welcome to request committer access by emailing support@thoughtbot.com.

Existing contributors become a maintainer by taking on more work and responsibility.