Skip to content

Commit

Permalink
chore: move configure-aws-credentials to outer action
Browse files Browse the repository at this point in the history
  • Loading branch information
strophy committed Nov 1, 2023
1 parent aa370af commit 631602b
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 13 deletions.
7 changes: 0 additions & 7 deletions .github/actions/nodejs/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,18 +17,11 @@ runs:
shell: bash
run: npm config set audit false

- name: Configure AWS credentials and bucket region
uses: aws-actions/configure-aws-credentials@v4
if: contains(runner.name, 'ubuntu-platform')
with:
aws-region: eu-west-1

- name: Cache NPM build artifacts (S3 bucket cache)
uses: everpcpc/actions-cache@v1
if: contains(runner.name, 'ubuntu-platform')
with:
bucket: multi-runner-linux-x64-platform-cache-ui535z23
endpoint: s3.eu-west-1.amazonaws.com
root: actions-cache
path: |
.yarn/unplugged
Expand Down
6 changes: 0 additions & 6 deletions .github/actions/rust/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,12 +31,6 @@ runs:
echo "PROTOC=${HOME}/.local/bin/protoc" >> $GITHUB_ENV
export PATH="${PATH}:${HOME}/.local/bin"
- name: Configure AWS credentials and bucket region
uses: aws-actions/configure-aws-credentials@v4
if: contains(runner.name, 'ubuntu-platform')
with:
aws-region: eu-west-1

- name: Cache Rust build artifacts (S3 bucket cache)
uses: everpcpc/actions-cache@v1
if: contains(runner.name, 'ubuntu-platform')
Expand Down
6 changes: 6 additions & 0 deletions .github/workflows/tests-build-js.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,12 @@ jobs:
username: ${{ secrets.DOCKERHUB_USERNAME }}
password: ${{ secrets.DOCKERHUB_TOKEN }}

- name: Configure AWS credentials and bucket region
uses: aws-actions/configure-aws-credentials@v4
if: contains(runner.name, 'ubuntu-platform')
with:
aws-region: eu-west-1

- name: Setup Node.JS
uses: ./.github/actions/nodejs

Expand Down

0 comments on commit 631602b

Please sign in to comment.