doc: Add "ci" prefix to CONTRIBUTING.md #17688

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:patch-1 changing 1 files +2 −2

hebasto commented at 9:53 AM on December 7, 2019: member
It seems our maintainers like ci prefix for commits and PRs:
```
git log | grep 'ci:'
```
and

So let's document it.
DrahtBot added the label Docs on Dec 7, 2019
MarcoFalke commented at 2:54 PM on December 7, 2019: member

Oh, I wasn't aware that build is used for ci changes as well. I saw ci mostly as a slightly more specific tests or qa.
hebasto commented at 3:21 PM on December 7, 2019: member
@MarcoFalke

Oh, I wasn't aware that build is used for ci changes as well. I saw ci mostly as a slightly more specific tests or qa.

Does it mean something like this:
```
  - `test`, `qa` or `ci` for changes to the unit tests, QA tests or CI code
```
?
laanwj commented at 11:32 AM on December 11, 2019: member

yes, CI refers to tests (it's a generalized "travis"), metadata such as .travis.yml and the things under the ci/ directory

(of course, this overlaps with "build" often, the categorization is by no means exclusive)
MarcoFalke added the label Brainstorming on Dec 11, 2019
doc: Add ci prefix to CONTRIBUTING.md 19267cbc82
hebasto force-pushed on Dec 15, 2019
hebasto commented at 7:31 PM on December 15, 2019: member

@MarcoFalke

I saw ci mostly as a slightly more specific tests or qa.

Done.
fanquake renamed this:
~~doc: Add "ci" component/area prefix to CONTRIBUTING.md~~
doc: Add "ci" prefix to CONTRIBUTING.md
on Jan 4, 2020
fanquake approved
fanquake commented at 6:22 AM on January 4, 2020: member

ACK 19267cbc82c1a09b37a0266b7bd930e9a62d1288 - this looks ok
fanquake referenced this in commit 9a4cc52d73 on Jan 4, 2020
fanquake merged this on Jan 4, 2020
fanquake closed this on Jan 4, 2020
hebasto deleted the branch on Jan 4, 2020
sidhujag referenced this in commit 485f5ab138 on Jan 4, 2020
sidhujag referenced this in commit 84ff22b1f0 on Nov 10, 2020
DrahtBot locked this on Feb 15, 2022

Contributors

Labels