diff options
author | Christopher Baines <mail@cbaines.net> | 2024-12-18 17:43:32 +0000 |
---|---|---|
committer | Christopher Baines <mail@cbaines.net> | 2025-01-04 08:32:41 +0000 |
commit | 321edcf0744a8895690579e8a5b09b66c75d102c (patch) | |
tree | 0733dcf0e749ca467856cc789223805ce1a54f89 /doc | |
parent | ddbb59f9bc5eb6dfaf1fc24881bb62f11960d664 (diff) |
doc: contributing: Add guidance on reverting commits.
Particularly to cover the case where things might need to happen with the
build farms if changes are reverted.
* doc/contributing.texi (Commit Access): Add guidance on reverting commits.
Change-Id: Iba320b76b0927b693c75054b5473a50bdd77c7ee
Diffstat (limited to 'doc')
-rw-r--r-- | doc/contributing.texi | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/doc/contributing.texi b/doc/contributing.texi index d4784de452..c94ae940fa 100644 --- a/doc/contributing.texi +++ b/doc/contributing.texi @@ -2945,6 +2945,20 @@ implicitly blame them, which is unhelpful. Accountability comes from a consensus about the problem, learning from it and improving processes so that it's less likely to reoccur. +@subsubsection Reverting commits + +Like normal commits, the commit message should state why the changes are +being made, which in this case would be why the commits are being +reverted. + +If the changes are being reverted because they led to excessive number +of packages being affected, then a decision should be made whether to +allow the build farms to build the changes, or whether to avoid +this. For the bordeaux build farm, commits can be ignored by adding them +to the @code{ignore-commits} list in the +@code{build-from-guix-data-service} record, found in the bayfront +machine configuration. + @subsection Commit Revocation In order to reduce the possibility of mistakes, committers will have |