Skip to content

Commit edf80d2

Browse files
peffgitster
authored andcommitted
ci: deprecate ci/config/allow-ref script
Now that we have the CI_BRANCHES mechanism, there is no need for anybody to use the ci/config/allow-ref mechanism. In the long run, we can hopefully remove it and the whole "config" job, as it consumes CPU and adds to the end-to-end latency of the whole workflow. But we don't want to do that immediately, as people need time to migrate until the CI_BRANCHES change has made it into the workflow file of every branch. So let's issue a warning, which will appear in the "annotations" section below the workflow result in GitHub's web interface. And let's remove the sample allow-refs script, as we don't want to encourage anybody to use it. Signed-off-by: Jeff King <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent 21c82dc commit edf80d2

File tree

2 files changed

+6
-30
lines changed

2 files changed

+6
-30
lines changed

.github/workflows/main.yml

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,10 +31,13 @@ jobs:
3131
name: check whether CI is enabled for ref
3232
run: |
3333
enabled=yes
34-
if test -x config-repo/ci/config/allow-ref &&
35-
! config-repo/ci/config/allow-ref '${{ github.ref }}'
34+
if test -x config-repo/ci/config/allow-ref
3635
then
37-
enabled=no
36+
echo "::warning::ci/config/allow-ref is deprecated; use CI_BRANCHES instead"
37+
if ! config-repo/ci/config/allow-ref '${{ github.ref }}'
38+
then
39+
enabled=no
40+
fi
3841
fi
3942
4043
skip_concurrent=yes

ci/config/allow-ref.sample

Lines changed: 0 additions & 27 deletions
This file was deleted.

0 commit comments

Comments
 (0)