Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update CI actions to use checkout@V4 to resolve node.js warning #24528

Merged
merged 1 commit into from
Mar 5, 2024
Merged

Update CI actions to use checkout@V4 to resolve node.js warning #24528

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions .github/workflows/CI.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ jobs:
username: ${{ github.actor }}
password: ${{ secrets.github_token }}
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- name: make check
run: |
./util/buildRelease/smokeTest chpl
Expand All @@ -39,7 +39,7 @@ jobs:
username: ${{ github.actor }}
password: ${{ secrets.github_token }}
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- name: make frontend-docs
run: |
make frontend-docs
Expand All @@ -61,7 +61,7 @@ jobs:
username: ${{ github.actor }}
password: ${{ secrets.github_token }}
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- name: make mason
# Use a quickstart config to keep it from running to long
# While there, run a make check in that config for more coverage
Expand All @@ -76,7 +76,7 @@ jobs:
username: ${{ github.actor }}
password: ${{ secrets.github_token }}
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- name: make test-dyno-with-asserts
run: |
CHPL_HOME=$PWD make DYNO_ENABLE_ASSERTIONS=1 test-dyno -j`util/buildRelease/chpl-make-cpu_count`
Expand All @@ -99,7 +99,7 @@ jobs:
username: ${{ github.actor }}
password: ${{ secrets.github_token }}
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- name: set llvm_none make check
run: |
CHPL_LLVM=none ./util/buildRelease/smokeTest chpl
Expand All @@ -111,7 +111,7 @@ jobs:
username: ${{ github.actor }}
password: ${{ secrets.github_token }}
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
with:
fetch-depth: 100000
- name: Set ownership
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/build-CI-container.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ jobs:
packages: write
steps:
- name: Checkout Code
uses: actions/checkout@v3
uses: actions/checkout@v4
- name: Login to GitHub Container Registry
uses: docker/[email protected]
with:
Expand Down
62 changes: 31 additions & 31 deletions .github/workflows/email.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,44 +5,44 @@ on:
pull_request_target:
types:
- closed
jobs:

jobs:
send_email:
# run this workflow step only when the PR is merged
if: github.event.pull_request.merged == true
runs-on: ubuntu-latest
# run this workflow step only when the PR is merged
if: github.event.pull_request.merged == true
runs-on: ubuntu-latest
steps:
- name: print git events
run: cat "$GITHUB_EVENT_PATH"
run: cat "$GITHUB_EVENT_PATH"
- name: print GitHub context
env:
GITHUB_CONTEXT: ${{ toJson(github) }}
run: echo "$GITHUB_CONTEXT"
run: echo "$GITHUB_CONTEXT"
# This workflow step will parse github env. The parsed env will be used in the email body.
- name: get commits payload
id: build-payload
env:
PR_LINK: ${{github.event.pull_request._links.html.href}}
run: |
echo "LINK=$(echo $PR_LINK )" >> $GITHUB_ENV
run: |
echo "LINK=$(echo $PR_LINK )" >> $GITHUB_ENV
echo "AUTHOR=$(echo ${{github.event.pull_request.head.user.login}})" >> $GITHUB_ENV
echo "COMPARE_URL= $( echo '${{github.event.repository.html_url}}/compare/${{github.event.pull_request.base.sha}}...${{github.event.pull_request.head.sha}}' )" >> $GITHUB_ENV
echo "FILES_CHANGED=$(echo '$PR_LINK/files' )" >> $GITHUB_ENV
echo "COMPARE_URL= $( echo '${{github.event.repository.html_url}}/compare/${{github.event.pull_request.base.sha}}...${{github.event.pull_request.head.sha}}' )" >> $GITHUB_ENV
echo "FILES_CHANGED=$(echo '$PR_LINK/files' )" >> $GITHUB_ENV

- name: checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
# To get git diff on the files that were changed in the PR checkout with fetch-depth 2.
with:
fetch-depth: 0
# 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.
fetch-depth: 0
# 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.
# (2) Get the files changed on the merge commit using the git log --name-status command.
- name: Get merge log
id: merge-log
env:
MERGE_SHA: ${{github.event.pull_request.merge_commit_sha}}
run: |
run: |
merge_log=$(git show -s --format=%B $MERGE_SHA )
echo "$merge_log"
echo "$merge_log"
echo 'MERGE<<EOF' >> $GITHUB_ENV
echo "$merge_log" >> $GITHUB_ENV
echo 'EOF' >> $GITHUB_ENV
Expand All @@ -52,12 +52,12 @@ jobs:
diff_max_lines=1000
diff=$(echo "$diff" | awk "NR <= $diff_max_lines; NR > $diff_max_lines { print \"(diff output truncated at $diff_max_lines lines)\"; exit }")

echo "$diff"
echo "$diff"
echo 'DIFF<<EOF' >> $GITHUB_ENV
echo "$diff" >> $GITHUB_ENV
echo 'EOF' >> $GITHUB_ENV
- name: Send mail
echo 'EOF' >> $GITHUB_ENV

- name: Send mail
uses: dawidd6/action-send-mail@v3
with:
# Required mail server address if not connection_url:
Expand All @@ -66,7 +66,7 @@ jobs:
# Optional whether this connection use TLS (default is true if server_port is 465)
secure: true
# Optional (recommended): mail server username:
username: ${{ secrets.MAIL_USERNAME}}
username: ${{ secrets.MAIL_USERNAME}}
# Optional (recommended) mail server password:
password: ${{secrets.MAIL_PASSWORD}}
# Required mail subject:
Expand All @@ -76,18 +76,18 @@ jobs:
# Required sender full name (address can be skipped):
from: ${{env.AUTHOR}}
body: |
Branch: ${{github.ref}}
Revision: ${{ github.event.pull_request.merge_commit_sha }}
Author: ${{ env.AUTHOR}}
Link: ${{github.event.pull_request._links.html.href}}
Log Message:
${{env.MERGE}}

Branch: ${{github.ref}}
Revision: ${{ github.event.pull_request.merge_commit_sha }}
Author: ${{ env.AUTHOR}}
Link: ${{github.event.pull_request._links.html.href}}
Log Message:
${{env.MERGE}}

Compare: ${{env.COMPARE_URL}}

Diff:
${{env.DIFF}}
Diff:
${{env.DIFF}}
${{github.event.pull_request.diff_url}}
# Optional priority: 'high', 'normal' (default) or 'low'
priority: low
Loading