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

refactor: port ZRC20 contract to v2 #281

Merged
merged 54 commits into from
Aug 5, 2024
Merged
Show file tree
Hide file tree
Changes from 50 commits
Commits
Show all changes
54 commits
Select commit Hold shift + click to select a range
5b5cc4b
access control for erc20 custody new
skosito Jul 23, 2024
f1cf789
cleanup erc20 custody new events and errors
skosito Jul 23, 2024
8b9669e
access control for gateway evm
skosito Jul 23, 2024
1851b89
move connector events to interface
skosito Jul 23, 2024
ba0f25f
add tss address to connector and fix uniswap integration test
skosito Jul 23, 2024
3f4990c
tests for tss sender in connector
skosito Jul 23, 2024
a686f66
remove access control todos from zevm contracts
skosito Jul 23, 2024
58160e8
generate
skosito Jul 23, 2024
0f8ab79
missing access control
skosito Jul 23, 2024
72e1200
init v2 folder
skosito Jul 24, 2024
8fafcbd
use npm
skosito Jul 24, 2024
99aaebb
moving files
skosito Jul 24, 2024
8e5b78f
moving files
skosito Jul 24, 2024
2964b06
move upgrade test
skosito Jul 24, 2024
3eac2ca
connector tests
skosito Jul 24, 2024
c0dbd8c
wip
skosito Jul 24, 2024
394e7c5
forge fmt
skosito Jul 24, 2024
09be45c
move echidna
skosito Jul 24, 2024
6672b9e
using submodule for forge-std
skosito Jul 24, 2024
2ea0a59
forge install: openzeppelin-contracts-upgradeable
skosito Jul 24, 2024
a8ec2b1
use submodules
skosito Jul 24, 2024
3a57e9d
move existing to v1 folder
skosito Jul 26, 2024
a3bf3f2
Merge branch 'main' into v1-v2-split
skosito Jul 29, 2024
731aa1e
yarn generate for v1
skosito Jul 29, 2024
3d6a870
cleanup
skosito Jul 29, 2024
3d736e5
conflicts
skosito Jul 29, 2024
0ec2a72
try to fix build action
skosito Jul 29, 2024
b334516
fixing v1 actions
skosito Jul 29, 2024
307a455
fix test v1
skosito Jul 29, 2024
d00c832
add paths to v1 actions
skosito Jul 29, 2024
b8db384
forge fmt
skosito Jul 29, 2024
56ef158
add test v2 and cleanup v1 actions
skosito Jul 29, 2024
9a7dd16
fix
skosito Jul 29, 2024
28af3e6
fix
skosito Jul 29, 2024
dd4cbeb
remove build v1 action and rename test actions steps
skosito Jul 29, 2024
2a01ed1
feat: port localnet to v2 (#269)
skosito Jul 29, 2024
4c013a2
readme and test fixes
skosito Jul 29, 2024
f401c98
lint v2
skosito Jul 29, 2024
8af5d05
enable coverage and slither on v2
skosito Jul 29, 2024
545201b
remove broadcast folder
skosito Jul 29, 2024
5231707
fix PR comments
skosito Jul 30, 2024
f4c1c7d
merge main
skosito Jul 30, 2024
640e67d
refactor: v2 bindings (#270)
skosito Jul 30, 2024
6835971
refactor: remove new suffix from v2 contracts (#272)
skosito Jul 31, 2024
5f2ea3c
feat: add missing access control (#277)
skosito Aug 1, 2024
ad2f4d9
modify zrc20
skosito Aug 1, 2024
a7897cd
generate
skosito Aug 1, 2024
48bf0de
add unit tests for zrc20
skosito Aug 1, 2024
b5a5576
generate
skosito Aug 1, 2024
a06e946
generate
skosito Aug 1, 2024
5a3892e
PR comment
skosito Aug 2, 2024
38081fb
Merge branch 'main' into zrc-20-fixes
skosito Aug 2, 2024
ca92f50
generate
skosito Aug 2, 2024
364c3ef
improve test a bit
skosito Aug 5, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
Original file line number Diff line number Diff line change
@@ -1,26 +1,32 @@
name: Generated Files are Updated
name: Generated Files are Updated (V1)

on:
push:
branches:
- main
paths:
- 'v1/**'
pull_request:
branches:
- "*"
paths:
- 'v1/**'
types:
- synchronize
- opened
- reopened
- ready_for_review

defaults:
run:
working-directory: ./v1

jobs:
generate:
runs-on: ubuntu-latest
steps:
- name: Checkout code
uses: actions/checkout@v4
with:
submodules: recursive

- name: Setup Node.js
uses: actions/setup-node@v4
Expand All @@ -40,9 +46,6 @@ jobs:
tar -zxvf geth-alltools-linux-amd64-1.11.5-a38f4108.tar.gz
sudo mv geth-alltools-linux-amd64-1.11.5-a38f4108/abigen /usr/local/bin/

- name: Install Foundry
uses: foundry-rs/foundry-toolchain@v1

- name: Generate Go packages and typechain-types
run: |
yarn generate
Expand Down
65 changes: 65 additions & 0 deletions .github/workflows/generated-files_v2.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,65 @@
name: Generated Files are Updated (V2)

on:
push:
branches:
- main
paths:
- 'v2/**'
pull_request:
branches:
- "*"
paths:
- 'v2/**'
types:
- synchronize
- opened
- reopened
- ready_for_review

defaults:
run:
working-directory: ./v2

jobs:
generate:
runs-on: ubuntu-latest
steps:
- name: Checkout code
uses: actions/checkout@v4
with:
submodules: recursive

- name: Setup Node.js
uses: actions/setup-node@v4
with:
node-version: "21.1.0"
registry-url: "https://registry.npmjs.org"

- name: Install Foundry
uses: foundry-rs/foundry-toolchain@v1

- name: Install dependencies
run: |
sudo apt-get update
sudo apt-get install -y jq unzip
yarn install

- name: Install specific version of aibgen
run: |
wget https://gethstore.blob.core.windows.net/builds/geth-alltools-linux-amd64-1.11.5-a38f4108.tar.gz
tar -zxvf geth-alltools-linux-amd64-1.11.5-a38f4108.tar.gz
sudo mv geth-alltools-linux-amd64-1.11.5-a38f4108/abigen /usr/local/bin/

- name: Generate Go packages and typechain-types
run: |
yarn generate

- name: Check for changes
run: |
if git diff --exit-code --ignore-space-change --ignore-all-space --ignore-cr-at-eol -- pkg typechain-types; then
echo "Generated Go files are up-to-date."
else
echo "::error::Generated files are not up-to-date. Please run 'yarn generate' locally and commit any changes."
exit 1
fi
15 changes: 9 additions & 6 deletions .github/workflows/lint.yaml → .github/workflows/lint_v1.yaml
Original file line number Diff line number Diff line change
@@ -1,37 +1,40 @@
name: Lint TS/JS
name: Lint TS/JS (V1)

on:
push:
branches:
- main
paths:
- 'v1/**'
pull_request:
branches:
- "*"
paths:
- 'v1/**'
types:
- synchronize
- opened
- reopened
- ready_for_review

defaults:
run:
working-directory: ./v1

jobs:
lint:
runs-on: ubuntu-latest

steps:
- name: Checkout Repository
uses: actions/checkout@v3
with:
submodules: recursive

- name: Setup Node.js
uses: actions/setup-node@v3
with:
node-version: "18"
registry-url: "https://registry.npmjs.org"

- name: Install Foundry
uses: foundry-rs/foundry-toolchain@v1

- name: Install Dependencies
run: yarn install

Expand Down
26 changes: 21 additions & 5 deletions .github/workflows/build.yaml → .github/workflows/lint_v2.yaml
Original file line number Diff line number Diff line change
@@ -1,34 +1,50 @@
name: Build
name: Lint TS/JS/Sol (V2)

on:
push:
branches:
- main
paths:
- 'v2/**'
pull_request:
branches:
- "*"
paths:
- 'v2/**'
types:
- synchronize
- opened
- reopened
- ready_for_review

defaults:
run:
working-directory: ./v2

jobs:
lint:
runs-on: ubuntu-latest

steps:
- name: Checkout Repository
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
submodules: recursive

- name: Setup Node.js
uses: actions/setup-node@v3
with:
node-version: "18"
node-version: "21.1.0"
registry-url: "https://registry.npmjs.org"

- name: Install Foundry
uses: foundry-rs/foundry-toolchain@v1

- name: Install Dependencies
run: yarn install

- name: Build
run: yarn build
- name: Lint JavaScript/TypeScript
run: yarn lint

- name: Lint Solidity
run: forge fmt --check
Original file line number Diff line number Diff line change
@@ -1,28 +1,27 @@
name: Publish to NPM
name: Publish to NPM (V1)

on:
release:
types: [published]

defaults:
run:
working-directory: ./v1

jobs:
publish:
runs-on: ubuntu-latest

steps:
- name: Checkout Repository
uses: actions/checkout@v3
with:
submodules: recursive

- name: Setup Node.js
uses: actions/setup-node@v3
with:
node-version: "18"
registry-url: "https://registry.npmjs.org"

- name: Install Foundry
uses: foundry-rs/foundry-toolchain@v1

- name: Install Dependencies
run: yarn install

Expand Down
Original file line number Diff line number Diff line change
@@ -1,18 +1,26 @@
name: Slither
name: Slither (V2)

on:
push:
branches:
- main
paths:
- 'v2/**'
pull_request:
branches:
- "*"
paths:
- 'v2/**'
types:
- synchronize
- opened
- reopened
- ready_for_review

defaults:
run:
working-directory: ./v2

jobs:
slither:
runs-on: ubuntu-latest
Expand All @@ -22,14 +30,14 @@ jobs:

steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4
with:
submodules: recursive

- name: Install Node.js
uses: actions/setup-node@v2
with:
node-version: "18"
node-version: "21.1.0"

- name: Install Foundry
uses: foundry-rs/foundry-toolchain@v1
Expand All @@ -38,15 +46,15 @@ jobs:
run: yarn install

- name: Build project
run: yarn build
run: forge build

- name: Run Slither
uses: crytic/slither-action@main
id: slither
continue-on-error: true
with:
sarif: results.sarif
node-version: "18"
node-version: "21.1.0"
fail-on: none

- name: Upload SARIF file
Expand Down
20 changes: 10 additions & 10 deletions .github/workflows/test.yaml → .github/workflows/test_v1.yaml
Original file line number Diff line number Diff line change
@@ -1,45 +1,45 @@
name: Test
name: Test (V1)

on:
push:
branches:
- main
paths:
- 'v1/**'
pull_request:
branches:
- "*"
paths:
- 'v1/**'
types:
- synchronize
- opened
- reopened
- ready_for_review

defaults:
run:
working-directory: ./v1

jobs:
test:
runs-on: ubuntu-latest

steps:
- name: Checkout Repository
uses: actions/checkout@v4
with:
submodules: recursive

- name: Setup Node.js
uses: actions/setup-node@v4
with:
node-version: "18.0.0"
registry-url: "https://registry.npmjs.org"

- name: Install Foundry
uses: foundry-rs/foundry-toolchain@v1

- name: Install Dependencies
run: yarn install

- name: Build project
run: yarn build

- name: Test (foundry)
run: yarn test:foundry


- name: Test (hardhat)
run: yarn test
Loading
Loading