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.

Sign up for GitHub

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

Jump to bottom

L2 public testnet deployment and related fixes #751

Merged
merged 43 commits into from
Nov 23, 2022
Merged

L2 public testnet deployment and related fixes #751

merged 43 commits into from
Nov 23, 2022

Conversation

pcarranzav
Copy link
Member

No description provided.

abarmat and others added 30 commits September 27, 2022 11:48
@abarmat
fix: use path.join to manage path concat
5e36583
@abarmat
feat: add publish and mint to the CLI
0ca43ff
@pcarranzav
chore: add reports for recent audits (#710)
45ce81e
@tmigone
chore: run GitHub actions when it makes sense (#707)
b6219d3 
Signed-off-by: Tomás Migone <[email protected]>
@tmigone
chore: provide sane defaults for graph config (#721)
04cfd3a 
Signed-off-by: Tomás Migone <[email protected]>
@tmigone
chore: remove rinkeby and kovan references (#729)
fdc0bae
@tmigone
feat: add optional expression logging to GRE (#728)
f1c5fe7 
* feat: add optional expression logging to GRE

Closes: #702
Signed-off-by: Tomás Migone <[email protected]>
@pcarranzav
test: print gas report, and ensure evm is killed
0b7055a
@pcarranzav
Merge pull request #732 from graphprotocol/pcv/test-gas-cat-report
3f16e5a 
test: print gas report, and ensure evm is killed
@tmigone
feat(gre): add hardhat-secure-accounts to GRE (#696)
7e5ac05 
* chore: update GRE to use secure accounts

Signed-off-by: Tomás Migone <[email protected]>
@tmigone
chore: update arbitrum sdk to 3.0-rc1
598ecc3 
Signed-off-by: Tomás Migone <[email protected]>
@tmigone
fix(task): accept ownership only if pending owner matches
a7ba535 
Signed-off-by: Tomás Migone <[email protected]>
@tmigone
feat(task): split unpause task into protocol and bridge tasks
0aed5eb 
Signed-off-by: Tomás Migone <[email protected]>
@tmigone
feat(e2e): run L1 and L2 together in e2e tests
a3fa719 
Signed-off-by: Tomás Migone <[email protected]>
@tmigone
feat(e2e): fetch deployment data from testnodes
ad11bdf 
Signed-off-by: Tomás Migone <[email protected]>
@tmigone
chore: update README.md
e9cab36 
Signed-off-by: Tomás Migone <[email protected]>
@tmigone
fix(e2e): allow skipping bridge tests
7eb7ba2 
Signed-off-by: Tomás Migone <[email protected]>
@tmigone
fix(e2e): update github action to multi layer e2e tests
1436a0c 
Signed-off-by: Tomás Migone <[email protected]>
@tmigone
fix(e2e): bump gas limit for l2 scenario txs
d9977b5 
Signed-off-by: Tomás Migone <[email protected]>
@tmigone
chore(e2e): run nitro addressbook setup only on localnitro networks
6a9a999 
Signed-off-by: Tomás Migone <[email protected]>
@tmigone
fix: allow e2e to run only on L2
9729f43 
Signed-off-by: Tomás Migone <[email protected]>
@tmigone
fix(e2e): use fixed gasLimit for CI send to L2
85c2964 
Signed-off-by: Tomás Migone <[email protected]>
@tmigone
fix(e2e): use fixed gasLimit for redeem in CI
ee8c7f3 
Signed-off-by: Tomás Migone <[email protected]>
@tmigone
e2e: add bridge e2e tests (#720)
967e085 
* feat(e2e): run L1 and L2 together in e2e tests

Signed-off-by: Tomás Migone <[email protected]>
@tmigone
fix: gateways now unpaused on e2e
ae1d70b 
Signed-off-by: Tomás Migone <[email protected]>
@abarmat
Merge branch 'dev' into l2-public-testnet
60afb9e
@tmigone @abarmat
chore: fix automerge wrong import
b1be78b 
Signed-off-by: Tomás Migone <[email protected]>
@abarmat
fix: remove unused task folder
ccaf653
@abarmat
fix: wrong imports and params
dff1845
@tmigone
fix: additional post merge fixes
5f1a3c4 
Signed-off-by: Tomás Migone <[email protected]>
tmigone and others added 5 commits November 3, 2022 16:03
@tmigone
fix: update nitro task for nitro v2.0.8
32fc123 
Signed-off-by: Tomás Migone <[email protected]>
@abarmat
chore: add deployed addresses for L2 public testnet
5daa3c1
@tmigone
fix: minor tweaks to e2e
fb40819 
Signed-off-by: Tomás Migone <[email protected]>
@abarmat
v1.17.0
3605081 
- feat(gre): add hardhat-secure-accounts to GRE (#696)
- feat: add publish and mint to the CLI
- feat: refactor GRE to support L1 and L2 simultaneously
- fix: prevent underflow when subgraphs go under the minimum signal threshold
@tmigone
chore: bump npm package version to v2.0.0-testnet
1dd6f06 
Signed-off-by: Tomás Migone <[email protected]>
@codecov
Copy link

codecov bot commented Nov 23, 2022 •
edited
Loading

Codecov Report

Base: 91.59% // Head: 91.59% // No change to project coverage 👍

Coverage data is based on head (51c0441) compared to base (d8fed24).
Patch has no changes to coverable lines.

❗ Current head 51c0441 differs from pull request most recent head aa85104. Consider uploading reports for the commit aa85104 to get more accurate results

Additional details and impacted files 
@@              Coverage Diff               @@
##           pcv/l2-bridge     #751   +/-   ##
==============================================
  Coverage          91.59%   91.59%           
==============================================
  Files                 42       42           
  Lines               1999     1999           
  Branches             361      361           
==============================================
  Hits                1831     1831           
  Misses               168      168
Flag Coverage Δ
unittests 91.59% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

abarmat
abarmat approved these changes Nov 23, 2022
View reviewed changes
Copy link
Contributor

@abarmat abarmat left a comment

Choose a reason for hiding this comment

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

🥇 🥇 🥇

@pcarranzav pcarranzav merged commit d35acb6 into pcv/l2-bridge Nov 23, 2022
@pcarranzav pcarranzav deleted the l2-public-testnet branch November 23, 2022 22:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@abarmat abarmat abarmat approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pcarranzav @abarmat @tmigone