Skip to content

Commit 8b030fb

Browse files
authoredMay 28, 2024··
chore: Fixed CI jobs (#2215)
1 parent d0a481d commit 8b030fb

File tree

2 files changed

+28
-14
lines changed

2 files changed

+28
-14
lines changed
 

‎.github/workflows/ci-workflow.yml

+22-13
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
11
name: Node Agent CI
22

3-
on: [push, pull_request, workflow_dispatch]
3+
on:
4+
push:
5+
pull_request:
6+
workflow_dispatch:
47

58
env:
69
# Enable versioned runner quiet mode to make CI output easier to read:
@@ -40,8 +43,9 @@ jobs:
4043
lint:
4144
needs:
4245
- should_run
43-
if: needs.should_run.outputs.javascript_changed == 'true' ||
44-
needs.should_run.outputs.deps_changed == 'true'
46+
if: github.event_name == 'workflow_dispatch' ||
47+
(needs.should_run.outputs.javascript_changed == 'true' ||
48+
needs.should_run.outputs.deps_changed == 'true')
4549
runs-on: ubuntu-latest
4650

4751
strategy:
@@ -62,8 +66,9 @@ jobs:
6266
ci:
6367
needs:
6468
- should_run
65-
if: needs.should_run.outputs.javascript_changed == 'true' ||
66-
needs.should_run.outputs.deps_changed == 'true'
69+
if: github.event_name == 'workflow_dispatch' ||
70+
(needs.should_run.outputs.javascript_changed == 'true' ||
71+
needs.should_run.outputs.deps_changed == 'true')
6772
runs-on: ubuntu-latest
6873

6974
strategy:
@@ -85,8 +90,9 @@ jobs:
8590
unit:
8691
needs:
8792
- should_run
88-
if: needs.should_run.outputs.javascript_changed == 'true' ||
89-
needs.should_run.outputs.deps_changed == 'true'
93+
if: github.event_name == 'workflow_dispatch' ||
94+
(needs.should_run.outputs.javascript_changed == 'true' ||
95+
needs.should_run.outputs.deps_changed == 'true')
9096
runs-on: ubuntu-latest
9197

9298
strategy:
@@ -113,8 +119,9 @@ jobs:
113119
integration:
114120
needs:
115121
- should_run
116-
if: needs.should_run.outputs.javascript_changed == 'true' ||
117-
needs.should_run.outputs.deps_changed == 'true'
122+
if: github.event_name == 'workflow_dispatch' ||
123+
(needs.should_run.outputs.javascript_changed == 'true' ||
124+
needs.should_run.outputs.deps_changed == 'true')
118125
runs-on: ubuntu-latest
119126

120127
env:
@@ -153,8 +160,9 @@ jobs:
153160
versioned-internal:
154161
needs:
155162
- should_run
156-
if: needs.should_run.outputs.javascript_changed == 'true' ||
157-
needs.should_run.outputs.deps_changed == 'true'
163+
if: github.event_name == 'workflow_dispatch' ||
164+
(needs.should_run.outputs.javascript_changed == 'true' ||
165+
needs.should_run.outputs.deps_changed == 'true')
158166
runs-on: ${{ github.ref == 'refs/heads/main' && vars.NR_RUNNER || 'ubuntu-latest' }}
159167

160168
strategy:
@@ -203,8 +211,9 @@ jobs:
203211
versioned-external:
204212
needs:
205213
- should_run
206-
if: needs.should_run.outputs.javascript_changed == 'true' ||
207-
needs.should_run.outputs.deps_changed == 'true'
214+
if: github.event_name == 'workflow_dispatch' ||
215+
(needs.should_run.outputs.javascript_changed == 'true' ||
216+
needs.should_run.outputs.deps_changed == 'true')
208217
runs-on: ${{ github.ref == 'refs/heads/main' && vars.NR_RUNNER || 'ubuntu-latest' }}
209218

210219
strategy:

‎.github/workflows/compatibility-report.yml

+6-1
Original file line numberDiff line numberDiff line change
@@ -90,13 +90,17 @@ jobs:
9090
arch: amd64
9191
cache: enable
9292
- run: |
93-
nrversions -v -r agent -R ${DOCS_TARGET} 2>docs-status.log
93+
nrversions -v -r agent -R ./docs/${DOCS_TARGET} 2>docs-status.log
9494
9595
# Upload generated artifacts for potential debugging purposes.
9696
- uses: actions/upload-artifact@v4
9797
with:
9898
name: docs-status.log
9999
path: docs-status.log
100+
- uses: actions/upload-artifact@v4
101+
with:
102+
name: compatibility-requirements-nodejs-agent.mdx
103+
path: docs/${{ env.DOCS_TARGET }}
100104

101105
# Generate the new PR to update the doc in the repo.
102106
- run: |
@@ -109,4 +113,5 @@ jobs:
109113
branch: "nodejs-compatibility-report/auto-update"
110114
delete-branch: true
111115
base: develop
116+
path: docs
112117
labels: "documentation"

0 commit comments

Comments
 (0)
Please sign in to comment.