Skip to content

Commit 4309b87

Browse files
ci: remove labeler and simplify change detection (#9751)
**Description** Created detect-changes.yml with centralized skip patterns and refactored all 8 workflows: - New workflow (detect-changes.yml): Uses dorny/paths-filter and centralizes skip patterns in one place (docs, images, contrib, various asset extensions) - Updated 8 workflows: All now call detect-changes and use needs.changes.outputs.code == 'true' instead of the previous should-skip flag - Deleted the old check-changes.yml Benefits: - Skip patterns defined once, not scattered across 8 workflows - Cleaner PR status checks (no intermediate check-changes job) - More robust change detection via dorny/paths-filter vs shell script - Same functionality, less overhead Commits: e3aa1fb (removed labeler) and 9db1fec6f (consolidated change detection) **Checklist** - [x] The PR title follows the [Conventional Commits](https://www.conventionalcommits.org/en/v1.0.0/#summary) syntax, leading with `fix:`, `feat:`, `chore:`, `ci:`, etc. --------- Co-authored-by: Claude Haiku 4.5 <noreply@anthropic.com>
1 parent bddaa4b commit 4309b87

12 files changed

Lines changed: 73 additions & 217 deletions

.github/labeler.yml

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

.github/workflows/check-changes.yml

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

.github/workflows/ci-dgraph-core-tests.yml

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,13 +17,12 @@ permissions:
1717
contents: read
1818

1919
jobs:
20-
check-changes:
21-
uses: ./.github/workflows/check-changes.yml
20+
changes:
21+
uses: ./.github/workflows/detect-changes.yml
2222

2323
dgraph-core-tests:
24-
needs: check-changes
25-
if:
26-
github.event.pull_request.draft == false && needs.check-changes.outputs.should-skip != 'true'
24+
needs: changes
25+
if: github.event.pull_request.draft == false && needs.changes.outputs.code == 'true'
2726
runs-on: blacksmith-8vcpu-ubuntu-2404
2827
timeout-minutes: 60
2928
steps:

.github/workflows/ci-dgraph-fuzz.yml

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,13 +15,12 @@ permissions:
1515
contents: read
1616

1717
jobs:
18-
check-changes:
19-
uses: ./.github/workflows/check-changes.yml
18+
changes:
19+
uses: ./.github/workflows/detect-changes.yml
2020

2121
fuzz-test:
22-
needs: check-changes
23-
if:
24-
github.event.pull_request.draft == false && needs.check-changes.outputs.should-skip != 'true'
22+
needs: changes
23+
if: github.event.pull_request.draft == false && needs.changes.outputs.code == 'true'
2524
runs-on: blacksmith-4vcpu-ubuntu-2404
2625
timeout-minutes: 10
2726
steps:

.github/workflows/ci-dgraph-integration2-tests.yml

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,13 +15,12 @@ permissions:
1515
contents: read
1616

1717
jobs:
18-
check-changes:
19-
uses: ./.github/workflows/check-changes.yml
18+
changes:
19+
uses: ./.github/workflows/detect-changes.yml
2020

2121
dgraph-integration2-tests:
22-
needs: check-changes
23-
if:
24-
github.event.pull_request.draft == false && needs.check-changes.outputs.should-skip != 'true'
22+
needs: changes
23+
if: github.event.pull_request.draft == false && needs.changes.outputs.code == 'true'
2524
runs-on: blacksmith-16vcpu-ubuntu-2404
2625
timeout-minutes: 90
2726
steps:

.github/workflows/ci-dgraph-ldbc-tests.yml

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,13 +15,12 @@ permissions:
1515
contents: read
1616

1717
jobs:
18-
check-changes:
19-
uses: ./.github/workflows/check-changes.yml
18+
changes:
19+
uses: ./.github/workflows/detect-changes.yml
2020

2121
dgraph-ldbc-tests:
22-
needs: check-changes
23-
if:
24-
github.event.pull_request.draft == false && needs.check-changes.outputs.should-skip != 'true'
22+
needs: changes
23+
if: github.event.pull_request.draft == false && needs.changes.outputs.code == 'true'
2524
runs-on: blacksmith-4vcpu-ubuntu-2404
2625
timeout-minutes: 10
2726
steps:

.github/workflows/ci-dgraph-load-tests.yml

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,13 +15,12 @@ permissions:
1515
contents: read
1616

1717
jobs:
18-
check-changes:
19-
uses: ./.github/workflows/check-changes.yml
18+
changes:
19+
uses: ./.github/workflows/detect-changes.yml
2020

2121
dgraph-load-tests:
22-
needs: check-changes
23-
if:
24-
github.event.pull_request.draft == false && needs.check-changes.outputs.should-skip != 'true'
22+
needs: changes
23+
if: github.event.pull_request.draft == false && needs.changes.outputs.code == 'true'
2524
runs-on: blacksmith-4vcpu-ubuntu-2404
2625
timeout-minutes: 30
2726
steps:

.github/workflows/ci-dgraph-systest-tests.yml

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,13 +17,12 @@ permissions:
1717
contents: read
1818

1919
jobs:
20-
check-changes:
21-
uses: ./.github/workflows/check-changes.yml
20+
changes:
21+
uses: ./.github/workflows/detect-changes.yml
2222

2323
dgraph-systest-tests:
24-
needs: check-changes
25-
if:
26-
github.event.pull_request.draft == false && needs.check-changes.outputs.should-skip != 'true'
24+
needs: changes
25+
if: github.event.pull_request.draft == false && needs.changes.outputs.code == 'true'
2726
runs-on: blacksmith-8vcpu-ubuntu-2404
2827
timeout-minutes: 60
2928
steps:

.github/workflows/ci-dgraph-tests-arm64.yml

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,13 +15,12 @@ permissions:
1515
contents: read
1616

1717
jobs:
18-
check-changes:
19-
uses: ./.github/workflows/check-changes.yml
18+
changes:
19+
uses: ./.github/workflows/detect-changes.yml
2020

2121
dgraph-tests:
22-
needs: check-changes
23-
if:
24-
github.event.pull_request.draft == false && needs.check-changes.outputs.should-skip != 'true'
22+
needs: changes
23+
if: github.event.pull_request.draft == false && needs.changes.outputs.code == 'true'
2524
runs-on: blacksmith-4vcpu-ubuntu-2404-arm
2625
timeout-minutes: 60
2726
steps:

.github/workflows/ci-dgraph-vector-tests.yml

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,13 +17,12 @@ permissions:
1717
contents: read
1818

1919
jobs:
20-
check-changes:
21-
uses: ./.github/workflows/check-changes.yml
20+
changes:
21+
uses: ./.github/workflows/detect-changes.yml
2222

2323
dgraph-vector-tests:
24-
needs: check-changes
25-
if:
26-
needs.check-changes.outputs.should-skip != 'true' && github.event.pull_request.draft == false
24+
needs: changes
25+
if: needs.changes.outputs.code == 'true' && github.event.pull_request.draft == false
2726
runs-on: blacksmith-16vcpu-ubuntu-2404
2827
timeout-minutes: 60
2928
steps:

0 commit comments

Comments
 (0)