From 414927043c235d666e28755b3068dae69633bb96 Mon Sep 17 00:00:00 2001 From: Amir Zarrinkafsh Date: Wed, 1 Jul 2020 16:07:27 +1000 Subject: [PATCH] [CI] Ignore CI/CD and notify for forked master branches (#1166) There is an issue with codecov where if a PR is made from a forked master branch this manages to assume itself as the default codecov branches, this in turn causes coverage synchronisation issues for our repo. --- .github/probot.js | 26 ++++++++++++++++++++++---- 1 file changed, 22 insertions(+), 4 deletions(-) diff --git a/.github/probot.js b/.github/probot.js index 51198afc..0f1f98ea 100644 --- a/.github/probot.js +++ b/.github/probot.js @@ -2,11 +2,11 @@ on('pull_request.opened') .filter( context => - context.payload.pull_request.head.label.slice(0, 9) === 'authelia:' + context.payload.pull_request.head.label.startsWith('authelia:') ) .filter( context => - context.payload.pull_request.head.ref.slice(0, 11) !== 'dependabot/' + !context.payload.pull_request.head.ref.startsWith('dependabot/') ) .comment(`## Artifacts These changes are published for testing on Buildkite and DockerHub. @@ -18,7 +18,11 @@ These changes are published for testing on Buildkite and DockerHub. on('pull_request.opened') .filter( context => - context.payload.pull_request.head.label.slice(0, 9) !== 'authelia:' + context.payload.pull_request.head.label.startsWith('authelia:') + ) + .filter( + context => + !context.payload.pull_request.head.label.endsWith(':master') ) .comment(`Thanks for choosing to contribute. We lint all PR's with golangci-lint, I may add a review to your PR with some suggestions. @@ -28,4 +32,18 @@ You are free to apply the changes if you're comfortable, alternatively you are w These changes once approved by a team member will be published for testing on Buildkite and DockerHub. ### Docker Container -* \`docker pull authelia/authelia:PR{{ pull_request.number }}\``) \ No newline at end of file +* \`docker pull authelia/authelia:PR{{ pull_request.number }}\``) + +// PR commentary for forked master branches +on('pull_request.opened') + .filter( + context => + context.payload.pull_request.head.label.startsWith('authelia:') + ) + .filter( + context => + context.payload.pull_request.head.label.endsWith(':master') + ) + .comment(`Thanks for choosing to contribute. It appears that you're submitting a PR from a forked master branch. + +This causes issues with codecov, please close this PR and re-submit your PR from a branch other than master.`) \ No newline at end of file