Skip to content

Commit

Permalink
update used gh actions ahead of set-output, node12 deprecation (#580)
Browse files Browse the repository at this point in the history
* update used gh actions ahead of set-output, node12 deprecation

* add changie log

* remove unneeded changelog

* Fix for changie that pointed to spark (#584)

---------

Co-authored-by: Neelesh Salian <nssalian@users.noreply.github.com>
  • Loading branch information
davidbloss and nssalian authored May 5, 2023
1 parent dd7cf32 commit a2c40f0
Show file tree
Hide file tree
Showing 4 changed files with 15 additions and 15 deletions.
14 changes: 7 additions & 7 deletions .github/workflows/integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -62,13 +62,13 @@ jobs:
steps:
- name: Check out the repository (non-PR)
if: github.event_name != 'pull_request_target'
uses: actions/checkout@v2
uses: actions/checkout@v3
with:
persist-credentials: false

- name: Check out the repository (PR)
if: github.event_name == 'pull_request_target'
uses: actions/checkout@v2
uses: actions/checkout@v3
with:
persist-credentials: false
ref: ${{ github.event.pull_request.head.sha }}
Expand All @@ -92,7 +92,7 @@ jobs:
- 'dev-requirements.txt'
- name: Generate integration test matrix
id: generate-matrix
uses: actions/github-script@v4
uses: actions/github-script@v6
env:
CHANGES: ${{ steps.get-changes.outputs.changes }}
with:
Expand Down Expand Up @@ -129,15 +129,15 @@ jobs:
steps:
- name: Check out the repository
if: github.event_name != 'pull_request_target'
uses: actions/checkout@v2
uses: actions/checkout@v3
with:
persist-credentials: false

# explicity checkout the branch for the PR,
# this is necessary for the `pull_request_target` event
- name: Check out the repository (PR)
if: github.event_name == 'pull_request_target'
uses: actions/checkout@v2
uses: actions/checkout@v3
with:
persist-credentials: false
ref: ${{ github.event.pull_request.head.sha }}
Expand Down Expand Up @@ -174,7 +174,7 @@ jobs:
DBT_TEST_USER_3: dbt_test_role_3
run: tox

- uses: actions/upload-artifact@v2
- uses: actions/upload-artifact@v3
if: always()
with:
name: logs
Expand All @@ -185,7 +185,7 @@ jobs:
id: date
run: echo "date=$(date +'%Y-%m-%dT%H_%M_%S')" >> $GITHUB_OUTPUT #no colons allowed for artifacts

- uses: actions/upload-artifact@v2
- uses: actions/upload-artifact@v3
if: always()
with:
name: integration_results_${{ matrix.python-version }}_${{ matrix.os }}_${{ matrix.adapter }}-${{ steps.date.outputs.date }}.csv
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/main-branch-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ jobs:
token: ${{ secrets.FISHTOWN_BOT_PAT }}

- name: Post failure to Slack
uses: ravsamhq/notify-slack-action@v1
uses: ravsamhq/notify-slack-action@v2
if: ${{ always() && !contains(steps.trigger-step.outputs.workflow-conclusion,'success') }}
with:
status: ${{ job.status }}
Expand Down
12 changes: 6 additions & 6 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ jobs:

steps:
- name: Check out the repository
uses: actions/checkout@v2
uses: actions/checkout@v3
with:
persist-credentials: false

Expand Down Expand Up @@ -80,7 +80,7 @@ jobs:

steps:
- name: Check out the repository
uses: actions/checkout@v2
uses: actions/checkout@v3
with:
persist-credentials: false

Expand All @@ -104,7 +104,7 @@ jobs:
id: date
run: echo "date=$(date +'%Y-%m-%dT%H_%M_%S')" >> $GITHUB_OUTPUT #no colons allowed for artifacts

- uses: actions/upload-artifact@v2
- uses: actions/upload-artifact@v3
if: always()
with:
name: unit_results_${{ matrix.python-version }}-${{ steps.date.outputs.date }}.csv
Expand All @@ -120,7 +120,7 @@ jobs:

steps:
- name: Check out the repository
uses: actions/checkout@v2
uses: actions/checkout@v3
with:
persist-credentials: false

Expand Down Expand Up @@ -156,7 +156,7 @@ jobs:
if [[ "$(ls -lh dist/)" == *"a1"* ]]; then export is_alpha=1; fi
echo "is_alpha=$is_alpha" >> $GITHUB_OUTPUT
- uses: actions/upload-artifact@v2
- uses: actions/upload-artifact@v3
with:
name: dist
path: dist/
Expand Down Expand Up @@ -188,7 +188,7 @@ jobs:
python -m pip install --upgrade wheel
python -m pip --version
- uses: actions/download-artifact@v2
- uses: actions/download-artifact@v3
with:
name: dist
path: dist/
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/release-branch-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ jobs:
token: ${{ secrets.FISHTOWN_BOT_PAT }}

- name: Post failure to Slack
uses: ravsamhq/notify-slack-action@v1
uses: ravsamhq/notify-slack-action@v2
if: ${{ always() && !contains(steps.trigger-step.outputs.workflow-conclusion,'success') }}
with:
status: ${{ job.status }}
Expand Down

0 comments on commit a2c40f0

Please sign in to comment.