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.

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

ci: update install action #28

Merged
merged 2 commits into from
Jul 24, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
16 changes: 8 additions & 8 deletions .github/workflows/run-nitro-test-node.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,35 +6,35 @@ on:

jobs:
run-with-defaults:
name: 'Run with defaults'
name: "Run with defaults"
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: ./run-nitro-test-node

run-with-token-bridge:
name: 'Run without token bridge deployment'
name: "Run without token bridge deployment"
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: ./run-nitro-test-node
with:
no-token-bridge: true

run-with-args:
name: 'Run with args'
name: "Run with args"
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: ./run-nitro-test-node
with:
args: --detach

run-with-simple:
name: 'Run in simple mode'
name: "Run in simple mode"
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: ./run-nitro-test-node
with:
no-simple: false
2 changes: 1 addition & 1 deletion node-modules/install/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4

- name: Install node_modules
uses: OffchainLabs/actions/node-modules/install@main
Expand Down
6 changes: 3 additions & 3 deletions node-modules/install/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ runs:
steps:
- name: Check cache for "node_modules"
id: cache
uses: martijnhols/actions-cache/check@v3
uses: actions/cache/restore@v4
with:
path: "**/node_modules"
key: ${{ inputs.cache-key }}
Expand All @@ -26,7 +26,7 @@ runs:

- name: Save "node_modules" to cache
if: steps.cache.outputs.cache-hit != 'true'
uses: martijnhols/actions-cache/save@v3
uses: actions/cache/save@v4
with:
path: "**/node_modules"
key: ${{ steps.cache.outputs.primary-key }}
key: ${{ inputs.cache-key }}
2 changes: 1 addition & 1 deletion node-modules/restore/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4

- name: Restore cache
uses: OffchainLabs/actions/node-modules/restore@main
Expand Down
38 changes: 19 additions & 19 deletions run-nitro-test-node/action.yml
Original file line number Diff line number Diff line change
@@ -1,45 +1,45 @@
name: Run Nitro Test Node
description: 'Checks out the Nitro repository and runs the local test node setup'
description: "Checks out the Nitro repository and runs the local test node setup"
inputs:
no-token-bridge:
required: false
default: 'false'
description: 'Whether to skip deploying the token bridge on the test node'
default: "false"
description: "Whether to skip deploying the token bridge on the test node"
no-l3-token-bridge:
required: false
default: 'false'
description: 'Whether to skip deploying the L3 token bridge on the test node'
default: "false"
description: "Whether to skip deploying the L3 token bridge on the test node"
no-simple:
required: false
default: 'true'
description: 'Whether to start the test node in simple mode'
default: "true"
description: "Whether to start the test node in simple mode"
args:
required: false
default: ''
description: 'Additional args that can be supplied to the test node script'
default: ""
description: "Additional args that can be supplied to the test node script"
nitro-testnode-ref:
required: false
default: 'release'
description: 'The nitro-testnode branch to use'
default: "release"
description: "The nitro-testnode branch to use"
l3-node:
required: false
default: 'false'
description: 'Whether to start an L3 node in addition to the L2 node'
default: "false"
description: "Whether to start an L3 node in addition to the L2 node"
nitro-contracts-branch:
required: false
description: 'The nitro-contracts branch to use'
description: "The nitro-contracts branch to use"
token-bridge-branch:
required: false
description: 'The token-bridge-contracts branch to use'
description: "The token-bridge-contracts branch to use"
runs:
using: 'composite'
using: "composite"
steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
repository: OffchainLabs/nitro-testnode
submodules: true
path: 'nitro-testnode'
path: "nitro-testnode"
ref: ${{ inputs.nitro-testnode-ref }}

- name: Start background nitro-testnode test-node.bash
Expand Down Expand Up @@ -80,4 +80,4 @@ runs:
- name: Wait for token bridge deployment
if: ${{ inputs.l3-node == 'true' && inputs.no-l3-token-bridge != 'true' }}
shell: bash
run: ${{ github.action_path }}/waitForTokenBridge.sh l2l3_network.json
run: ${{ github.action_path }}/waitForTokenBridge.sh l2l3_network.json