xref: /libCEED/CONTRIBUTING.md (revision 288c044332e33f37503f09b6484fec9d0a55fba1)
15d6bafb2Sjeremylt# libCEED: How to Contribute
274e0540eSJed Brown
399276e83SJed BrownContributions to libCEED are encouraged.
4ee7e4e48Sjeremylt<!---
574e0540eSJed BrownPlease use a pull request to the appropriate branch ('maint' for
674e0540eSJed Brownbackward-compatible bug fixes for the last stable release, 'master' for
774e0540eSJed Brownnew features and everything else).
8ee7e4e48Sjeremylt-->
999276e83SJed BrownPlease make your commits well-organized and
1099276e83SJed Brown[atomic](https://en.wikipedia.org/wiki/Atomic_commit#Atomic_commit_convention),
1199276e83SJed Brownusing `git rebase --interactive` as needed.  Check that tests
1299276e83SJed Brown(including "examples") pass using `make prove-all`.  If adding a new
1399276e83SJed Brownfeature, please add or extend a test so that your new feature is
1499276e83SJed Browntested.
1599276e83SJed Brown
1699276e83SJed BrownIn typical development, every commit should compile, be covered by the
1799276e83SJed Browntest suite, and pass all tests.  This improves the efficiency of
1899276e83SJed Brownreviewing and facilitates use of
1999276e83SJed Brown[`git bisect`](https://git-scm.com/docs/git-bisect).
2099276e83SJed Brown
2199276e83SJed BrownOpen an issue or RFC (request for comments) pull request to discuss
2299276e83SJed Brownany significant changes before investing time.  It is useful to create
2399276e83SJed Browna WIP (work in progress) pull request for any long-running development
2499276e83SJed Brownso that others can be aware of your work and help to avoid creating
2599276e83SJed Brownmerge conflicts.
2674e0540eSJed Brown
2774e0540eSJed BrownWrite commit messages for a reviewer of your pull request and for a
2874e0540eSJed Brownfuture developer (maybe you) that bisects and finds that a bug was
2999276e83SJed Brownintroduced in your commit.  The assumptions that are clear in your
3099276e83SJed Brownmind while committing are likely not in the mind of whomever (possibly
3199276e83SJed Brownyou) needs to understand it in the future.
3299276e83SJed Brown
3399276e83SJed BrownGive credit where credit is due using tags such as `Reported-by:
3499276e83SJed BrownHelpful User <helpful@example.com>`.  Please use a real name and email
3599276e83SJed Brownfor your author information (`git config user.name` and `user.email`).
3674e0540eSJed Brown
3774e0540eSJed BrownPlease avoid "merging from upstream" (like merging 'master' into your
3874e0540eSJed Brownfeature branch) unless there is a specific reason to do so, in which
3974e0540eSJed Browncase you should explain why in the merge commit.
4074e0540eSJed Brown[Rationale](https://lwn.net/Articles/328436/) from
4174e0540eSJed Brown[Junio](https://gitster.livejournal.com/42247.html) and
4274e0540eSJed Brown[Linus](http://yarchive.net/comp/linux/git_merges_from_upstream.html).
4374e0540eSJed Brown
4499276e83SJed BrownYou can use `make style` to help conform to coding conventions of the
4599276e83SJed Brownproject, but try to avoid mixing whitespace or formatting changes with
4699276e83SJed Browncontent changes (see atomicity above).
4799276e83SJed Brown
4874e0540eSJed BrownBy submitting a pull request, you are affirming the following.
4974e0540eSJed Brown
50*288c0443SJeremy L Thompson## [Developer's Certificate of Origin 1.1](https://developercertificate.org/)
5174e0540eSJed Brown
5274e0540eSJed BrownBy making a contribution to this project, I certify that:
5374e0540eSJed Brown
5474e0540eSJed Brown(a) The contribution was created in whole or in part by me and I
5574e0540eSJed Brown    have the right to submit it under the open source license
5674e0540eSJed Brown    indicated in the file; or
5774e0540eSJed Brown
5874e0540eSJed Brown(b) The contribution is based upon previous work that, to the best
5974e0540eSJed Brown    of my knowledge, is covered under an appropriate open source
6074e0540eSJed Brown    license and I have the right under that license to submit that
6174e0540eSJed Brown    work with modifications, whether created in whole or in part
6274e0540eSJed Brown    by me, under the same open source license (unless I am
6374e0540eSJed Brown    permitted to submit under a different license), as indicated
6474e0540eSJed Brown    in the file; or
6574e0540eSJed Brown
6674e0540eSJed Brown(c) The contribution was provided directly to me by some other
6774e0540eSJed Brown    person who certified (a), (b) or (c) and I have not modified
6874e0540eSJed Brown    it.
6974e0540eSJed Brown
7074e0540eSJed Brown(d) I understand and agree that this project and the contribution
7174e0540eSJed Brown    are public and that a record of the contribution (including all
7274e0540eSJed Brown    personal information I submit with it, including my sign-off) is
7374e0540eSJed Brown    maintained indefinitely and may be redistributed consistent with
7474e0540eSJed Brown    this project or the open source license(s) involved.
7574e0540eSJed Brown
7674e0540eSJed Brown
77