Skip to content

Commit

Permalink
Merge #598
Browse files Browse the repository at this point in the history
598: GitHub Actions: bump actions/checkout and actions/cache to v3 r=rtwalker a=rtwalker

We're seeing  a lot of deprecation warnings on our CI runs lately e.g. https://github.com/timescale/timescaledb-toolkit/actions/runs/3359573483

which point to this [blog post](https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/) about deprecations in some GitHub Actions commands.

Bumping the checkout and cache actions appears to make many of these warnings go away.

Co-authored-by: Ryan Walker <rwalker@timescale.com>
  • Loading branch information
bors[bot] and rtwalker authored Oct 31, 2022
2 parents d432bec + c02a5f9 commit 619563c
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 9 deletions.
14 changes: 7 additions & 7 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ jobs:

steps:
- name: Checkout Repository
uses: actions/checkout@v2
uses: actions/checkout@v3
with:
ref: ${{ github.event.pull_request.head.sha }}

Expand All @@ -58,7 +58,7 @@ jobs:
# TODO After the container image contains a primed target dir, is this still worth it?
# Only possible advantage is this one is per-pg-version but what's the impact?
- name: Cache cargo target dir
uses: actions/cache@v2
uses: actions/cache@v3
with:
path: target
key: ${{ runner.os }}-test-pg${{ matrix.pgversion }}-target-${{ hashFiles('Cargo.lock', '.github/workflows/ci.yml') }}
Expand Down Expand Up @@ -89,15 +89,15 @@ jobs:

steps:
- name: Checkout Repository
uses: actions/checkout@v2
uses: actions/checkout@v3
with:
ref: ${{ github.event.pull_request.head.sha }}

- name: chown Repository
run: chown -R postgres .

- name: Cache cargo target dir
uses: actions/cache@v2
uses: actions/cache@v3
with:
path: target
key: ${{ runner.os }}-test-crates-target-${{ hashFiles('Cargo.lock', '.github/workflows/ci.yml') }}
Expand All @@ -121,23 +121,23 @@ jobs:

steps:
- name: Checkout Repository
uses: actions/checkout@v2
uses: actions/checkout@v3
with:
ref: ${{ github.event.pull_request.head.sha }}

- name: chown Repository
run: chown -R postgres .

- name: Cache cargo target dir
uses: actions/cache@v2
uses: actions/cache@v3
with:
path: target
key: ${{ runner.os }}-test-updates-target-${{ hashFiles('Cargo.lock', '.github/workflows/ci.yml') }}
restore-keys: ${{ runner.os }}-test-crates-target-

# TODO Drop this.
- name: Cache old versions dir
uses: actions/cache@v2
uses: actions/cache@v3
with:
path: old-versions
key: ${{ runner.os }}-test-old-versions-${{ hashFiles('Cargo.lock', '.github/workflows/ci.yml') }}
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/clippy_rustfmt.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,15 +28,15 @@ jobs:

steps:
- name: Checkout Repository
uses: actions/checkout@v2
uses: actions/checkout@v3
with:
ref: ${{ github.event.pull_request.head.sha }}

- name: chown Repository
run: chown -R postgres .

- name: Cache cargo target dir
uses: actions/cache@v2
uses: actions/cache@v3
with:
path: target
key: ${{ runner.os }}-clippy-target-${{ hashFiles('Cargo.lock', '.github/workflows/clippy_rustfmt.yml') }}
Expand Down

0 comments on commit 619563c

Please sign in to comment.