Skip to content

Commit 5b805bd

Browse files
Fix dependabot labeler and workflows
1 parent 13658b8 commit 5b805bd

7 files changed

+9
-30
lines changed

.github/dependabot.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ updates:
1313
ignore:
1414
- dependency-name: "*"
1515
update-types: ["version-update:semver-patch"]
16-
open-pull-requests-limit: 4
16+
open-pull-requests-limit: 1
1717
labels:
1818
- "dependabot"
1919
- "test-guided-notebooks"

.github/workflows/dependabot-labeler.yaml

+4-4
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
# This workflow file adds the 'lgtm' and 'approved' labels to Dependabot PRs
2-
# This is done to ensure that the PRs that pass e2e are automatically merged/added to merge-queues by the CodeFlare bot
2+
# This is done to ensure that the PRs that pass required status checks are automatically merged by the CodeFlare bot
33
name: Dependabot Labeler
44

55
on:
6-
pull_request:
7-
types: [ labeled, synchronize, opened, reopened ]
6+
pull_request_target:
7+
branches: [ main ]
88

99
jobs:
1010
add-approve-lgtm-label:
@@ -24,4 +24,4 @@ jobs:
2424
run: |
2525
gh pr edit ${{ github.event.pull_request.number }} --add-label "lgtm" --add-label "approved"
2626
env:
27-
GITHUB_TOKEN: ${{ secrets.CODEFLARE_MACHINE_ACCOUNT_TOKEN }}
27+
GITHUB_TOKEN: ${{ secrets.GH_CLI_TOKEN }}

.github/workflows/e2e_tests.yaml

-12
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ name: e2e
33

44
on:
55
pull_request:
6-
types: [ labeled, synchronize, opened, reopened ]
76
branches:
87
- main
98
- 'release-*'
@@ -12,16 +11,6 @@ on:
1211
- '**.adoc'
1312
- '**.md'
1413
- 'LICENSE'
15-
push:
16-
branches:
17-
- main
18-
- 'release-*'
19-
paths-ignore:
20-
- 'docs/**'
21-
- '**.adoc'
22-
- '**.md'
23-
- 'LICENSE'
24-
merge_group:
2514

2615
concurrency:
2716
group: ${{ github.head_ref }}-${{ github.workflow }}
@@ -32,7 +21,6 @@ env:
3221

3322
jobs:
3423
kubernetes:
35-
if: contains(github.event.pull_request.labels.*.name, 'e2e') || github.event_name == 'merge_group'
3624
runs-on: ubuntu-20.04-4core-gpu
3725

3826
steps:

.github/workflows/guided_notebook_tests.yaml

+2-2
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ name: Guided notebooks tests
22

33
on:
44
pull_request:
5-
types: [ labeled, synchronize, opened, reopened ]
5+
branches: [ main ]
66

77
concurrency:
88
group: ${{ github.head_ref }}-${{ github.workflow }}
@@ -13,7 +13,7 @@ env:
1313

1414
jobs:
1515
verify-0_basic_ray:
16-
if: contains(github.event.pull_request.labels.*.name, 'test-guided-notebooks')
16+
if: ${{ contains(github.event.pull_request.labels.*.name, 'test-guided-notebooks') }}
1717
runs-on: ubuntu-20.04-4core
1818

1919
steps:

.github/workflows/pre-commit.yaml

-6
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,7 @@
11
name: Pre-commit
22
on:
3-
push:
4-
branches:
5-
- '**'
6-
tags-ignore:
7-
- 'v*'
83
pull_request:
94
workflow_dispatch:
10-
merge_group:
115

126
jobs:
137
precommit:

.github/workflows/ui_notebooks_test.yaml

+2-2
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ name: UI notebooks tests
22

33
on:
44
pull_request:
5-
types: [ labeled, synchronize, opened, reopened ]
5+
branches: [ main ]
66

77
concurrency:
88
group: ${{ github.head_ref }}-${{ github.workflow }}
@@ -13,7 +13,7 @@ env:
1313

1414
jobs:
1515
verify-3_widget_example:
16-
if: contains(github.event.pull_request.labels.*.name, 'test-guided-notebooks') || contains(github.event.pull_request.labels.*.name, 'test-ui-notebooks')
16+
if: ${{ contains(github.event.pull_request.labels.*.name, 'test-guided-notebooks') || contains(github.event.pull_request.labels.*.name, 'test-ui-notebooks') }}
1717
runs-on: ubuntu-20.04-4core
1818

1919
steps:

.github/workflows/unit-tests.yml

-3
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,8 @@
11
name: Python Tests
22

33
on:
4-
push:
5-
branches: [ main ]
64
pull_request:
75
branches: [ main ]
8-
merge_group:
96

107
jobs:
118
unit-tests:

0 commit comments

Comments
 (0)