aboutsummaryrefslogtreecommitdiff
path: root/CONTRIBUTING.md
AgeCommit message (Collapse)Author
2018-10-30added details about commit messagesMartin Erlandsson
2018-07-30doc: add note to contributor docs about warranted PR'sKarl-Johan Alm
2018-06-17Docs: Improve readability of "Squashing commits"wodry
It was not easy to read the comment lines for me because I was not sure whether the sentence ended with the line or not ("pull set commits"?). Therefore, dots had been invented and I have added them to signal the end of a sentence. Also begin New sentence with a capital letter. I guess, not all 'pick' words should be replaced by 'squash'? At least I found [https://www.digitalocean.com/community/tutorials/how-to-rebase-and-update-a-pull-request](this) rebase/squash documentation helpful, where is written that the first line should not be changed.
2018-05-04doc: Mention good first issue list in CONTRIBUTING.mdfanquake
2018-03-22Docs: Improve documentation on standard communication channelsJim Posen
More information about connection on IRC will hopefully help new contributors.
2017-11-20[docs] links to code style guidesSjors Provoost
2017-11-07Docs: Change formatting for sequence of stepsVivek Ganesan
A numbered list is used when the sequence of steps matters. A bulleted list is used to denote a set of items where the sequence does not matter. The workflow is a sequence and hence changed to a numbered list.
2017-10-28Fixed a couple small grammatical errors.Christian Gentry
1. "If a pull request is not to be considered for merging (yet), please prefix the ..." 2. If a particular commit references another issue, please add the reference. For example: `refs #1234` or `fixes #4321`.
2017-09-07Update CONTRIBUTRING.md to reduce unnecesarry review workloadJonas Schnelli
2017-09-05Add translation note to CONTRIBUTING.mdMeshCollider
2017-08-08Capitalize bullet points in CONTRIBUTING guideEvan Klitzke
English grammar dictates that these bullet points should be capitalized. This also makes the capitalization style consistent with the rest of the document, e.g. the "Decision Making Process" section.
2017-04-17[doc] Add blob about finding reviewers.Kalle Alm
2017-04-14[doc] Wording fixes in CONTRIBUTING.md.Kalle Alm
2017-03-23add missing spaces so that markdown recognizes headlineflack
2017-02-16Merge #9675: Fix typo and spelling inconsistency in CONTRIBUTING.mdWladimir J. van der Laan
36164fa Fix typo and spelling inconsistency in CONTRIBUTING.md (Koki Takahashi)
2017-02-16[Trivial] Update comments referencing main.cppCryptAxe
2017-02-09Fix typo and spelling inconsistency in CONTRIBUTING.mdKoki Takahashi
Fix spellings of GitHub Remove unnecessary changes Fix GitHub spelling on doc/translation_process.md
2017-01-18Docs: Update CONTRIBUTING.mdJohn Newbery
Update CONTRIBUTING.md to document the different components. Notably, trivial should only be used for PRs that do not change the code.
2016-10-13Add copyright/patent issues to possible NACK reasonsPeter Todd
Adding in response to a Slack discussion where someone was unclear on the fact that a NACK may be justified if code can't be accepted due to copyright/patent issues. For example, it would be reasonable and prudent to NACK a contribution of AGPL-licensed consensus code on the basis that the license terms are incompatible with the MIT license used by the rest of the codebase.
2016-09-25Merge #8771: CONTRIBUTING: Mention not to open several pullsWladimir J. van der Laan
faa91bc CONTRIBUTING: Mention not to open several pulls (MarcoFalke)
2016-09-23CONTRIBUTING: Mention not to open several pullsMarcoFalke
2016-09-22Mandatory copyright agreementAndrew Chow
All contributions to the repository will agree to use the MIT license
2016-06-23contributing.md: Fix formattingWill Binns
This commit contains two changes to CONTRIBUTING.md that: Fix line line lengths. There were several instances where line lengths were well over 80 characters. This commit adjusts them to make them conform to formatting best practices, to stay under 80 characters when possible. Adhere to consist use of quotes. There are a few instances where smart quotes are used (perhaps because it was pasted from a word processor). This commit replaces them with dumb quotes to keep it consistent with the quotation formatting found in the rest of the document.
2016-05-10[doc] Add basic git squash examplefanquake
2015-12-10Note that reviewers should mention the commit hash of the commits they reviewed.Patrick Strateman
2015-09-30Add PR title prefix for trivial changes [skip ci]paveljanik
2015-09-26Add CONTRIBUTING.mdBtcDrak