diff options
author | Boris Kolpackov <boris@codesynthesis.com> | 2024-10-30 12:24:30 +0200 |
---|---|---|
committer | Boris Kolpackov <boris@codesynthesis.com> | 2024-10-30 12:24:30 +0200 |
commit | fde758aa54324644b7378de2788691a2740ae2e3 (patch) | |
tree | 32fd584db8ce6496d8fb3a9c035e42175c970086 /doc | |
parent | 52b7690a7c6413378562a696dd225df242f1da6a (diff) |
Document examples of surprising GitHub CI behavior
Diffstat (limited to 'doc')
-rw-r--r-- | doc/manual.cli | 21 |
1 files changed, 19 insertions, 2 deletions
diff --git a/doc/manual.cli b/doc/manual.cli index 7ae8ca3..509b99d 100644 --- a/doc/manual.cli +++ b/doc/manual.cli @@ -797,7 +797,24 @@ commits are the same, which is only the case when the PR base branch can be fast-forwarded to head. Thankfully, GitHub provides a branch protection rule that prevents merging of -a PR with the head branch behind base. And enabling of this protection rule is -a prerequisite for this CI integration to work correctly. +a PR with the head branch behind base (we will refer to it as the +\i{head-behind-base} protection). Enabling of this protection rule is a +prerequisite for this CI integration to work correctly. + +Note, however, that even with the head-behind-base protection enabled, some of +the GitHub behavior can be counter-intuitive. + +For one, GitHub does not prevent the CI build from starting if this protection +rule is violated. While this integration checks the result of this protection +rule and does not start the build if the head is behind, the CI result may +already be available (if this head is shared with a branch push and/or another +PR), in which case GitHub will show it. So you may end up with a violated +head-behind-base protection but with a successful CI result. + +Another surprising consequent of the head sharing is the instantaneous +availability of the CI result, which may look suspicious. For example, if you +create a PR from a local feature branch, you may immediately see the +successful CI result because it is the same as for the branch push +to the feature branch. " |