Skip to content

feat: update to latest version of relayer-sdk and hardhat template#63

Open
poppyseedDev wants to merge 2 commits intomainfrom
upgrade-relayer
Open

feat: update to latest version of relayer-sdk and hardhat template#63
poppyseedDev wants to merge 2 commits intomainfrom
upgrade-relayer

Conversation

@poppyseedDev
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread .github/workflows/ci.yml
run: pnpm sdk:build

- name: Type check SDK
run: pnpm sdk:build
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

duplicate task ?

Comment thread .github/workflows/ci.yml
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v4
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing version SHA

Comment thread .github/workflows/ci.yml
uses: actions/checkout@v4

- name: Setup pnpm
uses: pnpm/action-setup@v4
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

idem

Comment thread .github/workflows/ci.yml
version: 9

- name: Setup Node.js
uses: actions/setup-node@v4
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

idem

Comment thread .github/workflows/ci.yml
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v4
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

idem

Comment thread .github/workflows/ci.yml
version: 9

- name: Setup Node.js
uses: actions/setup-node@v4
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

idem

Comment thread .github/workflows/ci.yml
NEXT_PUBLIC_PRIVY_APP_ID: ${{ secrets.NEXT_PUBLIC_PRIVY_APP_ID }}
steps:
- name: Checkout
uses: actions/checkout@v4
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

idem

Comment thread .github/workflows/ci.yml
uses: actions/checkout@v4

- name: Setup pnpm
uses: pnpm/action-setup@v4
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

idem

Comment thread .github/workflows/ci.yml
version: 9

- name: Setup Node.js
uses: actions/setup-node@v4
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

idem


// Query the KMSVerifier and InputVerifier contracts for their EIP712 domains
// This is necessary because these values differ between networks (Sepolia vs localhost)
const [kmsVerifierDomain, inputVerifierDomain] = await Promise.all([
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

avoid promise.all on public RPC nodes. It might trigger an error. Better call sequencially

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants