Skip to content

Commit 6b59247

Browse files
tzinskytzinsky
committed
Update CI actions to use checkout@V4 to resolve node.js warning
Github's CI migrated to a new version of node.js and this causes warnings on our CI jobs. The new version of the checkout action picks up this new version of node to resolve the warnings. Signed-off-by: Tim Zinsky <[email protected]>
1 parent 2b0b06f commit 6b59247

File tree

3 files changed

+38
-38
lines changed

3 files changed

+38
-38
lines changed

‎.github/workflows/CI.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
username: ${{ github.actor }}
2727
password: ${{ secrets.github_token }}
2828
steps:
29-
- uses: actions/checkout@v3
29+
- uses: actions/checkout@v4
3030
- name: make check
3131
run: |
3232
./util/buildRelease/smokeTest chpl
@@ -39,7 +39,7 @@ jobs:
3939
username: ${{ github.actor }}
4040
password: ${{ secrets.github_token }}
4141
steps:
42-
- uses: actions/checkout@v3
42+
- uses: actions/checkout@v4
4343
- name: make frontend-docs
4444
run: |
4545
make frontend-docs
@@ -61,7 +61,7 @@ jobs:
6161
username: ${{ github.actor }}
6262
password: ${{ secrets.github_token }}
6363
steps:
64-
- uses: actions/checkout@v3
64+
- uses: actions/checkout@v4
6565
- name: make mason
6666
# Use a quickstart config to keep it from running to long
6767
# While there, run a make check in that config for more coverage
@@ -76,7 +76,7 @@ jobs:
7676
username: ${{ github.actor }}
7777
password: ${{ secrets.github_token }}
7878
steps:
79-
- uses: actions/checkout@v3
79+
- uses: actions/checkout@v4
8080
- name: make test-dyno-with-asserts
8181
run: |
8282
CHPL_HOME=$PWD make DYNO_ENABLE_ASSERTIONS=1 test-dyno -j`util/buildRelease/chpl-make-cpu_count`
@@ -99,7 +99,7 @@ jobs:
9999
username: ${{ github.actor }}
100100
password: ${{ secrets.github_token }}
101101
steps:
102-
- uses: actions/checkout@v3
102+
- uses: actions/checkout@v4
103103
- name: set llvm_none make check
104104
run: |
105105
CHPL_LLVM=none ./util/buildRelease/smokeTest chpl
@@ -111,7 +111,7 @@ jobs:
111111
username: ${{ github.actor }}
112112
password: ${{ secrets.github_token }}
113113
steps:
114-
- uses: actions/checkout@v3
114+
- uses: actions/checkout@v4
115115
with:
116116
fetch-depth: 100000
117117
- name: Set ownership

‎.github/workflows/build-CI-container.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ jobs:
3434
packages: write
3535
steps:
3636
- name: Checkout Code
37-
uses: actions/checkout@v3
37+
uses: actions/checkout@v4
3838
- name: Login to GitHub Container Registry
3939
uses: docker/[email protected]
4040
with:

‎.github/workflows/email.yml

Lines changed: 31 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -5,44 +5,44 @@ on:
55
pull_request_target:
66
types:
77
- closed
8-
9-
jobs:
8+
9+
jobs:
1010
send_email:
11-
# run this workflow step only when the PR is merged
12-
if: github.event.pull_request.merged == true
13-
runs-on: ubuntu-latest
11+
# run this workflow step only when the PR is merged
12+
if: github.event.pull_request.merged == true
13+
runs-on: ubuntu-latest
1414
steps:
1515
- name: print git events
16-
run: cat "$GITHUB_EVENT_PATH"
16+
run: cat "$GITHUB_EVENT_PATH"
1717
- name: print GitHub context
1818
env:
1919
GITHUB_CONTEXT: ${{ toJson(github) }}
20-
run: echo "$GITHUB_CONTEXT"
20+
run: echo "$GITHUB_CONTEXT"
2121
# This workflow step will parse github env. The parsed env will be used in the email body.
2222
- name: get commits payload
2323
id: build-payload
2424
env:
2525
PR_LINK: ${{github.event.pull_request._links.html.href}}
26-
run: |
27-
echo "LINK=$(echo $PR_LINK )" >> $GITHUB_ENV
26+
run: |
27+
echo "LINK=$(echo $PR_LINK )" >> $GITHUB_ENV
2828
echo "AUTHOR=$(echo ${{github.event.pull_request.head.user.login}})" >> $GITHUB_ENV
29-
echo "COMPARE_URL= $( echo '${{github.event.repository.html_url}}/compare/${{github.event.pull_request.base.sha}}...${{github.event.pull_request.head.sha}}' )" >> $GITHUB_ENV
30-
echo "FILES_CHANGED=$(echo '$PR_LINK/files' )" >> $GITHUB_ENV
31-
29+
echo "COMPARE_URL= $( echo '${{github.event.repository.html_url}}/compare/${{github.event.pull_request.base.sha}}...${{github.event.pull_request.head.sha}}' )" >> $GITHUB_ENV
30+
echo "FILES_CHANGED=$(echo '$PR_LINK/files' )" >> $GITHUB_ENV
31+
3232
- name: checkout
33-
uses: actions/checkout@v3
33+
uses: actions/checkout@v4
3434
# To get git diff on the files that were changed in the PR checkout with fetch-depth 2.
3535
with:
36-
fetch-depth: 0
37-
# this step will (1) get the merge log for the merge_commit_sha and save it to the git actions EN for later use in the email body.
36+
fetch-depth: 0
37+
# this step will (1) get the merge log for the merge_commit_sha and save it to the git actions EN for later use in the email body.
3838
# (2) Get the files changed on the merge commit using the git log --name-status command.
3939
- name: Get merge log
4040
id: merge-log
4141
env:
4242
MERGE_SHA: ${{github.event.pull_request.merge_commit_sha}}
43-
run: |
43+
run: |
4444
merge_log=$(git show -s --format=%B $MERGE_SHA )
45-
echo "$merge_log"
45+
echo "$merge_log"
4646
echo 'MERGE<<EOF' >> $GITHUB_ENV
4747
echo "$merge_log" >> $GITHUB_ENV
4848
echo 'EOF' >> $GITHUB_ENV
@@ -52,12 +52,12 @@ jobs:
5252
diff_max_lines=1000
5353
diff=$(echo "$diff" | awk "NR <= $diff_max_lines; NR > $diff_max_lines { print \"(diff output truncated at $diff_max_lines lines)\"; exit }")
5454
55-
echo "$diff"
55+
echo "$diff"
5656
echo 'DIFF<<EOF' >> $GITHUB_ENV
5757
echo "$diff" >> $GITHUB_ENV
58-
echo 'EOF' >> $GITHUB_ENV
59-
60-
- name: Send mail
58+
echo 'EOF' >> $GITHUB_ENV
59+
60+
- name: Send mail
6161
uses: dawidd6/action-send-mail@v3
6262
with:
6363
# Required mail server address if not connection_url:
@@ -66,7 +66,7 @@ jobs:
6666
# Optional whether this connection use TLS (default is true if server_port is 465)
6767
secure: true
6868
# Optional (recommended): mail server username:
69-
username: ${{ secrets.MAIL_USERNAME}}
69+
username: ${{ secrets.MAIL_USERNAME}}
7070
# Optional (recommended) mail server password:
7171
password: ${{secrets.MAIL_PASSWORD}}
7272
# Required mail subject:
@@ -76,18 +76,18 @@ jobs:
7676
# Required sender full name (address can be skipped):
7777
from: ${{env.AUTHOR}}
7878
body: |
79-
80-
Branch: ${{github.ref}}
81-
Revision: ${{ github.event.pull_request.merge_commit_sha }}
82-
Author: ${{ env.AUTHOR}}
83-
Link: ${{github.event.pull_request._links.html.href}}
84-
Log Message:
85-
${{env.MERGE}}
79+
80+
Branch: ${{github.ref}}
81+
Revision: ${{ github.event.pull_request.merge_commit_sha }}
82+
Author: ${{ env.AUTHOR}}
83+
Link: ${{github.event.pull_request._links.html.href}}
84+
Log Message:
85+
${{env.MERGE}}
8686
8787
Compare: ${{env.COMPARE_URL}}
8888
89-
Diff:
90-
${{env.DIFF}}
89+
Diff:
90+
${{env.DIFF}}
9191
${{github.event.pull_request.diff_url}}
9292
# Optional priority: 'high', 'normal' (default) or 'low'
9393
priority: low

0 commit comments

Comments
 (0)