Skip to content

Commit fc5a411

Browse files
Copilotfregante
andauthored
Meta: CI workflow tweaks (#8839)
Co-authored-by: fregante <me@fregante.com>
1 parent d134639 commit fc5a411

4 files changed

Lines changed: 62 additions & 69 deletions

File tree

.github/workflows/conversation.yml

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
name: Conversation
2+
3+
on:
4+
pull_request_target:
5+
types: [opened, edited]
6+
issues:
7+
types: [opened, edited]
8+
9+
jobs:
10+
Title:
11+
runs-on: ubuntu-latest
12+
permissions:
13+
pull-requests: write
14+
issues: write
15+
steps:
16+
- uses: fregante/title-to-labels-action@v1
17+
with:
18+
token: ${{ github.token }}
19+
- uses: actions/checkout@v4
20+
- uses: fregante/keyword-formatter-action@v1
21+
with:
22+
token: ${{ github.token }}
23+
keywords-path: source/features

.github/workflows/labeler.yml

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

.github/workflows/pr.yml

Lines changed: 32 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,32 @@
1+
name: PR
2+
3+
on:
4+
pull_request_target:
5+
types: [closed]
6+
pull_request_review:
7+
types: [submitted]
8+
9+
jobs:
10+
reminder:
11+
name: Label Reminder
12+
if: github.event_name == 'pull_request_review' || github.event.pull_request.merged == true
13+
runs-on: ubuntu-latest
14+
permissions:
15+
contents: read
16+
pull-requests: write
17+
steps:
18+
- name: Check if PR has no labels
19+
id: check_labels
20+
uses: actions/github-script@v7
21+
with:
22+
result-encoding: string
23+
script: |
24+
const labels = context.payload.pull_request.labels;
25+
return labels.length === 0;
26+
27+
- name: Post comment if unlabeled, or fail on third party PRs
28+
if: steps.check_labels.outputs.result == 'true'
29+
uses: marocchino/sticky-pull-request-comment@v2
30+
with:
31+
GITHUB_TOKEN: ${{ github.token }}
32+
message: 'To maintainers: Please add labels to this PR'

.github/workflows/test.yml

Lines changed: 7 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -27,11 +27,13 @@ jobs:
2727
- run: sudo apt-get install ripgrep
2828
- run: npm ci
2929
- run: npm run vitest -- --exclude 'build/features*' --exclude '**/selectors*'
30-
- run: npm run vitest -- selectors
31-
# For some reason it fails on CI, but not locally
32-
# Mentioned in https://github.com/refined-github/refined-github/issues/7747
33-
if: false
34-
continue-on-error: true
30+
31+
# TODO: Restore
32+
# For some reason it fails on CI, but not locally
33+
# Mentioned in https://github.com/refined-github/refined-github/issues/7747
34+
# - run: npm run vitest -- selectors
35+
# if: false
36+
# continue-on-error: true
3537

3638
Verify:
3739
runs-on: ubuntu-latest
@@ -109,18 +111,3 @@ jobs:
109111
path: |
110112
distribution
111113
!./distribution/assets/webpack-stats.json
112-
113-
Safari:
114-
# Disabled because it's a waste of time
115-
if: false
116-
runs-on: macos-latest
117-
steps:
118-
- uses: actions/checkout@v4
119-
- uses: actions/setup-node@v4
120-
with:
121-
node-version-file: package.json
122-
cache: npm
123-
- run: npm ci
124-
- run: npm run build:bundle
125-
- uses: maxim-lobanov/setup-xcode@v1
126-
- run: npm run pack:safari

0 commit comments

Comments
 (0)