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
…#612)

* 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 <[email protected]>
# Conflicts:
#	.github/workflows/main-branch-tests.yml
#	.github/workflows/release-branch-tests.yml

Co-authored-by: David Bloss <[email protected]>
  • Loading branch information
emmyoop and davidbloss authored May 16, 2023
1 parent 7d0355d commit d1563fe
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 13 deletions.
14 changes: 7 additions & 7 deletions .github/workflows/integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -66,13 +66,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 @@ -95,7 +95,7 @@ jobs:
- 'tests/**'
- 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 @@ -133,15 +133,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 @@ -178,7 +178,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 @@ -189,7 +189,7 @@ jobs:
id: date
run: echo "::set-output name=date::$(date +'%Y-%m-%dT%H_%M_%S')" #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
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 "::set-output name=date::$(date +'%Y-%m-%dT%H_%M_%S')" #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 "::set-output name=is_alpha::$is_alpha"
- 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

0 comments on commit d1563fe

Please sign in to comment.