-
Notifications
You must be signed in to change notification settings - Fork 383
Develop #588
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
Open
krlosMata
wants to merge
350
commits into
main
Choose a base branch
from
develop
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Develop #588
Changes from all commits
Commits
Show all changes
350 commits
Select commit
Hold shift + click to select a range
6fee775
should work
invocamanman 7227ca1
lower not upgraded
invocamanman 1c63265
pr fixes
invocamanman f7f2b27
final fixes
invocamanman cd4d3dd
natspec
invocamanman a95247d
update commetns
invocamanman f652d12
update
invocamanman 4867e85
agglayer gateway consistent
invocamanman 02eb08b
comments
invocamanman 6ab36bb
udpate comment
invocamanman 131c3ea
fix: review remediations
ignasirv ad3a3c0
fix: update token wrapped bridge runs optimization
ignasirv 719eb0f
pr fixes
invocamanman bcfe27f
Merge pull request #517 from agglayer/feature/pr_fixes
krlosMata 2d2e397
fix: last remediation iteration
ignasirv 21ff5d0
small docs fixes
invocamanman 8219e40
Merge pull request #519 from agglayer/feature/informationals_fixes
krlosMata 8ef6061
Merge pull request #520 from agglayer/feature/pr_fixes
krlosMata e2a6846
Merge pull request #504 from agglayer/feature/outposts
krlosMata d9491b8
feat: Finished upgrade contracts to v12 script
ignasirv f1e4e04
feat: improvements
ignasirv c97353f
Move verification results to utils
ignasirv 3488111
feat: First iteration, simple approach
ignasirv ff95f94
Merge pull request #522 from agglayer/feature/upgrade-v12-script
krlosMata c8d0a42
Support for aggchainmanager transaction on 4_createRollup
ignasirv 71a7e0b
remediations
invocamanman 527adb5
Merge pull request #528 from agglayer/feature/v12_fixes
krlosMata b7e021e
all global owners
krlosMata cd6a497
fix: patch contracts verification script (#531)
tnobayashi dcc53ec
comments: check params
krlosMata 4af560f
Merge pull request #524 from agglayer/feature/initialize-tool-refactor
krlosMata fbd110c
PolygonRollupManager -> AgglayerManager
laisolizq 70c00f5
PolygonZkEVMGlobalExitRootV2 -> AgglayerGER
laisolizq 93ac621
PolygonZkEVMGlobalExitRootL2 -> LegacyAgglayerGERL2
laisolizq 15828bf
PolygonZkEVMBridgeV2 -> AgglayerBridge
laisolizq 5a76c92
BridgeL2SovereignChain -> AgglayerBridgeL2
laisolizq d794ad7
GlobalExitRootManagerL2SovereignChain -> AgglayerGERL2
laisolizq a19ed74
IBasePolygonZkEVMGlobalExitRoot -> IBaseLegacyAgglayerGER
laisolizq 0619fd9
PolygonZkEVMGlobalExitRootBaseStorage -> LegacyAgglayerGERBaseStorage
laisolizq 7aa0193
fix lint
laisolizq abc7f7a
rebase & update upgradeV12
laisolizq af10369
add test-vectors multisig-hash. update aggchainHash testvectors
krlosMata 2219e98
PolygonRollupManagerNotUpgraded -> AgglayerManagerNotUpgraded, AggLay…
laisolizq ecc7df9
allow aggchainECDSAMultig with isVanillaClient = false. Add batchData…
krlosMata 1746d41
fix linter
krlosMata 4e2270f
Merge pull request #529 from agglayer/feature/global-owners
krlosMata 427d89c
Merge pull request #532 from agglayer/feature/fix-create-rollup
krlosMata c6ad19a
review fixes
laisolizq c44ff1c
reorg v2 -> contracts
laisolizq c5c7bc1
Merge pull request #525 from agglayer/feature/renaming-agglayer
krlosMata 9002eb8
update typechain-types tests
laisolizq ecb19a1
Merge pull request #526 from agglayer/feature/reorg-v2
krlosMata 67ce44d
fix genesis bridgeLib + inconsistencies
krlosMata c2a988c
review copilot
krlosMata 3df541a
Merge pull request #536 from agglayer/feature/fix-1-create-genesis
krlosMata 1c00e15
feat: remove root check at migration flow
ignasirv 856c6e8
AG35-01
ignasirv c74c68f
AG35-04
ignasirv 2055377
AG35-05
ignasirv d504ceb
AG35-07
ignasirv 76b2b5d
AG35-08
ignasirv 2280478
#200
ignasirv cb917f7
fix comments aggchainFEP & aggchain_hash
krlosMata c3ccb67
add aggOracle report
krlosMata dc924c4
threshold aglayerGateway internal
krlosMata 1ce3c5c
Merge pull request #534 from agglayer/feature/fix-migration
krlosMata 3a10d04
fixes copilot & codex
krlosMata 3e6f73e
Merge pull request #538 from agglayer/feature/informational-remediations
krlosMata 26c2d38
Merge pull request #537 from agglayer/feature/audit-remediations
krlosMata 60f6077
renaming timelock
invocamanman a88bdb5
Merge pull request #542 from agglayer/feature/renaming_timelock
invocamanman f9a0d72
update deploy sovereign test
invocamanman c01773e
Merge pull request #527 from agglayer/feature/udpate_tools
invocamanman e05b928
fix upgradeV12 test
laisolizq 7c0924e
fix bridgeLib hardhatconfig
laisolizq 853352f
full upgrade
invocamanman c13c536
update test
invocamanman a14fbfa
works
invocamanman a3814a1
test passes
invocamanman 3fac87e
update test to test all timelock txs!
invocamanman b7dfd0f
importOZ info
invocamanman 95d852d
fix lint
invocamanman 5c354a7
etherscan veirifcation
invocamanman 62aeb83
delete tools
laisolizq 7714a84
Merge pull request #544 from agglayer/feature/full_upgrade_v12
krlosMata d1a1b7e
Merge pull request #543 from agglayer/feature/fix-upgradeV12
krlosMata 236cf04
compare genesis tool
ToniRamirezM a4e2dcd
feat: add Foundry support and enhance testing framework
JDawg287 53f9179
fix sovereign genesis tool
krlosMata 8f27a07
fix lint
krlosMata 848b278
Merge pull request #546 from agglayer/feature/fix-sovereign-genesis-tool
krlosMata 029f156
add final report v0.3.5
krlosMata 6482810
Merge pull request #549 from agglayer/feature/add-final-audit-v0.3.5
krlosMata 0d04d66
add tests tools
laisolizq c733514
Update CHANGELOG.md v12.1.2
laisolizq 63e8a08
Merge pull request #551 from agglayer/feature/changelog-v12.1.2
krlosMata 1c402f6
update compare genesis
invocamanman 8b7c75c
add script upgrade gerL2 etrog to sovereign
laisolizq 40cba32
add spearbits audit
krlosMata 248710c
update README upgrade gerL2
laisolizq 59ecf4e
Merge pull request #555 from agglayer/feature/v0.3.5-spearbits-audit
krlosMata af586c4
reviews changes
laisolizq 79e02cf
update README upgrade GERL2 to sovereign
laisolizq 36da2cd
Merge pull request #554 from agglayer/feature/upgrade-ger
krlosMata 76c7ed6
minor fixes
krlosMata 5cbb4e7
full delete comment
krlosMata 77e23c6
fix lint
krlosMata 896b37d
Merge pull request #557 from agglayer/feature/fix-fee-handling-and-sh…
invocamanman e9b6594
add test claim reentrancy
laisolizq ab370d7
test claim reentrancy, bridgeAsset, etc
laisolizq 13bb9a5
lint fixes
laisolizq 58689cb
update test with dif amounts
laisolizq 0f02ed2
fix: add constructor
tnobayashi 3c866d2
Merge pull request #558 from agglayer/tnobayashi/fix-deploy-aggoracle…
krlosMata 1a085d0
deployOutpost: delete not used example params
krlosMata eb47ff1
Merge pull request #559 from agglayer/feature/outpost-deployment-fix-…
krlosMata 49a1f19
rename test reentrancy with AgglayerBridge
laisolizq cb73eb5
fix path genesis test docker
laisolizq 692a856
Merge pull request #485 from agglayer/feature/test-claim-reentrancy
krlosMata bce91ea
Merge pull request #550 from agglayer/feature/update-test-docker-tools
krlosMata 6e10130
docs: update README
JDawg287 06a7fb2
Merge pull request #547 from agglayer/feature/foundry-compile
krlosMata fe3dfda
Remove boilerplate consts
alrevuelta 5b470ff
Cleanup old testnets
alrevuelta 3315b89
Move to constants file and replace everywhere
alrevuelta b992bd1
Merge pull request #561 from agglayer/cleanup-test-1
krlosMata 7410260
Merge pull request #562 from agglayer/cleanup-testnets
alrevuelta bc5012e
Cleanup upgrade to v2
alrevuelta 79e5678
minor fix
alrevuelta f078f95
revert mainnet folder
alrevuelta 5ee07e0
support previous genesis-base versions
krlosMata 50709ae
Merge pull request #564 from agglayer/remove-upgrade-v1-v2
alrevuelta a75ce7e
codex review
krlosMata cf163c3
Copilot review
krlosMata 546e3cd
fix linter
krlosMata d7190e7
Copilot: fix comment
krlosMata 672eb16
Merge pull request #545 from agglayer/feauture/compareGenesisTool
krlosMata 9f75ee5
Merge pull request #566 from agglayer/feature/fix-supported-bridges-n…
krlosMata 9861c04
script push docker all release
laisolizq 75dea94
Remove old unused tests
alrevuelta d0c33ae
Undo index and run with test
alrevuelta d45bc83
add detailed event, ongoing
invocamanman df85551
cursor comments
invocamanman 16ebc9c
bytecode opt
invocamanman cac6187
bytecode optimizations
invocamanman 1417025
update init approach
invocamanman 295c088
easies approach... for no
invocamanman 024e872
update test
invocamanman b2ec86f
set claim bytes32
invocamanman fdd84cf
update genesis scripts
invocamanman 19b4fb6
update bridge version
invocamanman 5d8c57a
fix udpate vanilla genesis
invocamanman ecdde63
fix commnets
invocamanman 54f79f0
udpate nonce
invocamanman 522bd5a
pr fixes
invocamanman 076f3c6
fix
invocamanman a59759c
test(foundry): remove unnecessary deployment methods
JDawg287 66d042a
chore: disable foundry lint on build
JDawg287 43ce0aa
feat: obsolete rollup type id script
JDawg287 1a3af8a
chore: add README for the obsolete rollup script
JDawg287 d17d57d
chore: add reminders
JDawg287 bc64d0e
Revert "chore: add reminders"
JDawg287 60be784
fix lint SC
krlosMata ef1739a
fix: refactor script to be feature complete
JDawg287 cfdd8a0
test: handle if the address is the proxy admin
JDawg287 6a662d9
fix: remove unused function
JDawg287 50f54b9
fix: remove unused lib
JDawg287 2ea6027
Merge pull request #572 from agglayer/cleanup2
krlosMata 4820ba0
Improve tool grantRoles --> manageRoles
krlosMata a4add91
Add timelock roles + review copilot
krlosMata 6fc5746
copilot review
krlosMata 1bd9f6d
copilot review 2
krlosMata 5bc5f44
Merge pull request #548 from agglayer/feature/obsolete-rollup
invocamanman 683c273
pure overriden functions
invocamanman ad93476
Merge pull request #573 from agglayer/fix/foundry-tests
invocamanman 0322b8f
remove bytecode storer in case it exists
invocamanman f27cc65
Merge pull request #568 from agglayer/feature/emit_detailed_events
invocamanman 92984ae
copilot review 3
krlosMata 7000168
copilot review 4
krlosMata 4c9de23
fix lint
krlosMata 6bb50e0
add script build docker all
laisolizq 84c180f
Merge pull request #569 from agglayer/feature/docker-all-push
krlosMata 4cef048
check json validity at the top of the scipt
krlosMata d23726a
Merge pull request #579 from agglayer/feature/improve-tool-manage-roles
krlosMata ab67049
upgrade script etrog -> sovereign
laisolizq b0dbcce
Add README & comments upgradeEtrogToSovereign
laisolizq cb805c3
add unsafeMode and more checks
laisolizq f3f1dd0
add check initialized version
laisolizq cc1142f
update with initialize setLBT
laisolizq 9ee0053
add initialize LBT script
laisolizq b6a70cf
update getLBT
laisolizq 246aad7
review changes
laisolizq 1911c7b
codex review
laisolizq b222c57
reviews
laisolizq bd202fe
updates script upgradeEtrogToSovereign
laisolizq 6bda16a
update upgrade etrog with prepare manifest
laisolizq 287593e
reviews fixes
laisolizq f937c67
paralelize getLBT tool + small improvements
krlosMata 8748166
copilot fixes
krlosMata 27058ca
second review copilot
krlosMata 23480ca
copilot review 3
krlosMata 3f4535c
fix lint
laisolizq 5580317
merge
laisolizq 0c6e0c7
mini fixes
laisolizq b45633b
add eth leaf LBT
laisolizq 79eb81c
add flag lbt path
laisolizq 6ea7ee9
mini fixes
laisolizq 4792852
new test
laisolizq a548687
WIP new approach upgrade etrog
laisolizq 424b160
review new upgrade etrog
laisolizq b9ad33b
update test upgrade etrog
laisolizq dc530b1
update shadow fork test
laisolizq 91176a1
WIP review
laisolizq 310d5a2
script check rollup versions
invocamanman 5a71add
fix contract adn script
invocamanman fc65972
udpate test!
invocamanman b415c87
test finished
invocamanman cf5ffcb
update scripts and readme
invocamanman e49c349
script upgrade bridge
invocamanman 0bffe17
udpate scripts
invocamanman e7126bd
small change
invocamanman 82b3f34
better UX
invocamanman ceed2ef
update LBT
invocamanman e973c5f
update script upgrade etrog
invocamanman 2af091f
git ignore
invocamanman 24ba635
finally
invocamanman 13d321f
add quick guide
invocamanman 2d160bc
done
invocamanman 3089dee
lint contract
invocamanman 3e89c3b
update readme
invocamanman b8f2861
udpate shadow fork
invocamanman ae543bb
update fork and LBT
invocamanman 986749e
update gitignore
invocamanman 6e10dc8
lint
invocamanman 87d395c
Merge pull request #533 from agglayer/feature/upgrade-etrog-sovereign
krlosMata 8c693b3
Merge branch 'develop' into feature/v12
krlosMata d6fff16
update changelog v12.2.1
laisolizq a38c614
Merge pull request #585 from agglayer/feature/v12-changelog
krlosMata a2ce05f
readme update
krlosMata 8ea5760
ObsoleteRollup tool
krlosMata fd9d4ac
fix linter. codex & copilot review
krlosMata b9b16b1
purge mode, improve fetching, improve README.md
krlosMata d994e29
fix eslint
krlosMata b4dcca9
Merge pull request #571 from agglayer/feature/tool-obsolete-rollups
krlosMata b6b4aa3
Merge pull request #586 from agglayer/feature/readme-update
krlosMata 077f34b
fix push docker release
laisolizq e30a478
Merge pull request #587 from agglayer/feature/fix-push-docker
krlosMata 97cad9a
Merge pull request #584 from agglayer/feature/v12
krlosMata e8b9316
fix deploy outpost proxiedTokensManager
laisolizq 7f1de4f
Merge pull request #590 from agglayer/fix/proxied-tokens-manager
krlosMata File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,2 @@ | ||
| # global owner | ||
| * @invocamanman | ||
|
|
||
| # src owners | ||
| /contracts/ @invocamanman @krlosMata @ignasirv @laisolizq | ||
| * @invocamanman @krlosMata @alrevuelta @laisolizq |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| ## PR Description | ||
|
|
||
| ## Checklist | ||
| - [ ] The CHANGELOG file has been updated. | ||
| - [ ] The release change information has been added in more detail to the documentation [repository](https://github.com/agglayer/protocol-team-docs). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,31 @@ | ||
| > This template is not meant to be automatically generated in GitHub, but it is useful to have it easily accessible when we create a release. | ||
|
|
||
| ## Contracts Versions | ||
| * ⚠️ `AgglayerManager`: ~~v0.0.0~~ -> v1.0.0 | ||
| * `AgglayerBridge`: v1.0.0 | ||
| * `AgglayerGER`: v1.0.0 | ||
| * `AgglayerGateway`: v1.0.0 | ||
| * `AgglayerBridgeL2`: v1.0.0 | ||
| * `AgglayerGERL2`: v1.0.0 | ||
| * `AggchainFEP`: v1.0.0 | ||
| * `AggchainECDSA`: v1.0.0 | ||
|
|
||
| ## 🚀 What's New | ||
| - [Brief description of new features] | ||
|
|
||
| ## ⚠️ Breaking Changes | ||
| - **Config File Changes**: [Describe any configuration file modifications] | ||
| - **API Changes**: [Document any breaking API modifications] | ||
| - **Deprecated Features**: [List any features being removed] | ||
|
|
||
| ## 📋 Configuration Updates | ||
| - **Config Diff**: [Link to diff showing configuration changes] | ||
| - **Updated Config File**: [Direct GitHub link to the exact config file version] | ||
| - **Migration Guide**: [Step-by-step instructions for updating existing configs] | ||
|
|
||
| ## ✅ Testing & Validation | ||
| - **Passing CI Run**: [Link to successful e2e test execution] | ||
| - **Test Coverage**: [Confirmation of test coverage for new features] | ||
|
|
||
| ## 📦 Full Changelog | ||
| [Auto-generated PR list and detailed changes] |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,4 @@ | ||
| blank_issues_enabled: true | ||
| pull_request_templates: | ||
| - name: Release PR | ||
| path: .github/PULL_REQUEST_TEMPLATE/release.md |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,34 @@ | ||
| # This workflow will do a clean install of node dependencies, build the source code and run tests across different versions of node | ||
| # For more information see: https://help.github.com/actions/language-and-framework-guides/using-nodejs-with-github-actions | ||
|
|
||
| name: Build-Docker all image | ||
|
|
||
| on: | ||
| push: | ||
| branches: [main, develop, feature/v12, feature/outposts] | ||
| pull_request: | ||
| branches: [main, develop, feature/v12, feature/outposts] | ||
| jobs: | ||
| build: | ||
| runs-on: ubuntu-latest | ||
|
|
||
| strategy: | ||
| matrix: | ||
| node-version: [22.x] | ||
|
|
||
| steps: | ||
| - name: Use Node.js ${{ matrix.node-version }} | ||
| uses: actions/setup-node@v3 | ||
| with: | ||
| node-version: ${{ matrix.node-version }} | ||
| - name: Checkout code | ||
| uses: actions/checkout@v3 | ||
| - name: Setup docker | ||
| run: | | ||
| npm i | ||
| sudo curl -L "https://github.com/docker/compose/releases/download/1.26.0/docker-compose-$(uname -s)-$(uname -m)" -o /usr/local/bin/docker-compose | ||
| sudo chmod +x /usr/local/bin/docker-compose | ||
| - name: Build docker | ||
| run: npm run dockerv2:contracts:all | ||
| - name: Test docker | ||
| run: npm run docker:tests | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,138 @@ | ||
| name: Push Docker all image | ||
|
|
||
| on: | ||
| release: | ||
| types: [created, published] | ||
|
|
||
| concurrency: | ||
| group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }} | ||
| cancel-in-progress: true | ||
|
|
||
| permissions: | ||
| contents: write | ||
| packages: write | ||
|
|
||
| env: | ||
| REGISTRY: ghcr.io | ||
| REGISTRY_IMAGE: ghcr.io/${{ github.repository }} | ||
| REGEX_IMAGE: ${{ github.repository }}:\\d+.\\d+.\\d+ | ||
|
|
||
| jobs: | ||
| build: | ||
| runs-on: ubuntu-latest | ||
| strategy: | ||
| fail-fast: false | ||
| matrix: | ||
| platform: | ||
| - linux/amd64 | ||
| - linux/arm64 | ||
| outputs: | ||
| IMAGE_NAME: ${{ steps.image_builder.outputs.IMAGE }} | ||
| TAGS: ${{ steps.meta.outputs.tags }} | ||
| VERSION: ${{ steps.meta.outputs.version }} | ||
| steps: | ||
|
|
||
| - name: Prepare | ||
| run: | | ||
| platform=${{ matrix.platform }} | ||
| echo "PLATFORM_PAIR=${platform//\//-}" >> $GITHUB_ENV | ||
|
|
||
| - name: Checkout | ||
| uses: actions/checkout@v4 | ||
|
|
||
| - name: setup | ||
| run: | | ||
| npm i | ||
| sudo curl -L "https://github.com/docker/compose/releases/download/1.26.0/docker-compose-$(uname -s)-$(uname -m)" -o /usr/local/bin/docker-compose | ||
| sudo chmod +x /usr/local/bin/docker-compose | ||
|
|
||
| - name: Docker meta | ||
| id: meta | ||
| uses: docker/metadata-action@v5 | ||
| with: | ||
| images: ${{ env.REGISTRY_IMAGE }} | ||
| tags: | | ||
| type=ref,event=branch,suffix=-all | ||
| type=ref,event=pr,suffix=-all | ||
| type=semver,pattern={{version}}-all | ||
| type=semver,pattern={{major}}.{{minor}}-all | ||
|
|
||
| - name: Image name builder | ||
| id: image_builder | ||
| run: | | ||
| IMAGE=$(jq -ecr '.tags | map(select(match("${{ env.REGEX_IMAGE }}", "i"))) | first| sub(":.*$";"")' <<< "$DOCKER_METADATA_OUTPUT_JSON") | ||
| echo "IMAGE=$IMAGE" >> $GITHUB_OUTPUT | ||
|
|
||
| - name: Set up QEMU | ||
| uses: docker/setup-qemu-action@v3 | ||
|
|
||
| - name: Set up Docker Buildx | ||
| uses: docker/setup-buildx-action@v3 | ||
|
|
||
| - name: Login to Docker Hub | ||
| uses: docker/login-action@v3 | ||
| with: | ||
| registry: ${{ env.REGISTRY }} | ||
| username: ${{ github.actor }} | ||
| password: ${{ secrets.GITHUB_TOKEN }} | ||
|
|
||
| - name: Build docker | ||
| run: npm run dockerv2:contracts:all | ||
| - name: Build and push by digest | ||
| id: build | ||
| uses: docker/build-push-action@v6 | ||
| with: | ||
| context: . | ||
| push: false | ||
| file: docker/Dockerfile | ||
| platforms: ${{ matrix.platform }} | ||
| labels: ${{ steps.meta.outputs.labels }} | ||
| cache-from: type=gha | ||
| cache-to: type=gha,mode=max | ||
| outputs: type=image,name=${{ steps.image_builder.outputs.IMAGE }},push-by-digest=true,push=true | ||
|
|
||
| - name: Export digest | ||
| run: | | ||
| mkdir -p /tmp/digests | ||
| digest="${{ steps.build.outputs.digest }}" | ||
| touch "/tmp/digests/${digest#sha256:}" | ||
|
|
||
| - name: Upload digest | ||
| uses: actions/upload-artifact@v4 | ||
| with: | ||
| name: digests-${{ env.PLATFORM_PAIR }} | ||
| path: /tmp/digests/* | ||
| if-no-files-found: error | ||
| retention-days: 1 | ||
|
|
||
| merge: | ||
| runs-on: ubuntu-latest | ||
| needs: | ||
| - build | ||
| steps: | ||
| - name: Download digests | ||
| uses: actions/download-artifact@v4 | ||
| with: | ||
| path: /tmp/digests | ||
| pattern: digests-* | ||
| merge-multiple: true | ||
|
|
||
| - name: Set up Docker Buildx | ||
| uses: docker/setup-buildx-action@v3 | ||
|
|
||
| - name: Login to Docker Hub | ||
| uses: docker/login-action@v3 | ||
| with: | ||
| registry: ${{ env.REGISTRY }} | ||
| username: ${{ github.actor }} | ||
| password: ${{ secrets.GITHUB_TOKEN }} | ||
|
|
||
| - name: Create manifest list and push | ||
| working-directory: /tmp/digests | ||
| run: | | ||
| docker buildx imagetools create $(jq -cRr 'split("\n") | map(gsub("\\s+";"")) | map("-t "+ .) | join(" ")' <<< '${{ needs.build.outputs.TAGS }}') \ | ||
| $(printf '${{ needs.build.outputs.IMAGE_NAME }}@sha256:%s ' *) | ||
|
|
||
| - name: Inspect image | ||
| run: | | ||
| docker buildx imagetools inspect ${{ needs.build.outputs.IMAGE_NAME }}:${{ needs.build.outputs.VERSION }} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,7 +1,5 @@ | ||
| contracts/verifiers/* | ||
| contracts/v2/utils/ClaimCompressor.sol | ||
| contracts/v2/previousVersions/* | ||
| contracts/v2/mocks/* | ||
| contracts/previousVersions/* | ||
| contracts/mocks/* | ||
| contracts/v2/periphery/* | ||
| contracts/v2/lib/LegacyZKEVMStateVariables.sol | ||
| contracts/periphery/* | ||
| contracts/lib/LegacyZKEVMStateVariables.sol |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Check warning
Code scanning / CodeQL
Workflow does not contain permissions Medium