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

main auditv2 merge #1782

Closed
wants to merge 89 commits into from
Closed

main auditv2 merge #1782

wants to merge 89 commits into from

Conversation

yorhodes
Copy link
Member

@yorhodes yorhodes commented Feb 8, 2023

Description

What's included in this PR?

Drive-by changes

Are there any minor or drive-by changes also included?

Related issues

  • Fixes #[issue number here]

Backward compatibility

Are these changes backward compatible?

Yes
No

Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling?

None
Yes

Testing

What kind of testing have these changes undergone?

None
Manual
Unit Tests

yorhodes and others added 30 commits December 26, 2022 07:53
* Small patch to CI exit condition

* Move instructions into yarn gas

* Update snapshot
* Improve docs and comments

* Cleanup interchain queries tests

* Update gas snapshot

* Improve middleware test coverage

* Add forge coverage report

* Update gas snapshot

* Add coverage report to PR comments

* Use latest version of coverage report action
* Verify mainnet3 and deployer fixes

* More fixes for testnet3

* Remove commented lines from verifier
* Update contracts with initialize owner changes

* Sdk updates

* Deploy ICAs and IQS to testnet3

* Deploy ICAs and IQS to mainnet2

* Update artifacts with names

* Fix celo IGP

* Redeploy middlewares to testnet3

* Fix alfajores verification

* Redeploy middlewares to mainnet2

* Use helloworld main

* Address pr comments

* Simplify expect violations

* Update gas snapshot
nambrot and others added 27 commits January 25, 2023 11:10
* Standardize matching list terms

* More consistent naming

* Use originDomain and destinationDomain

* Fix lint

* Undo test_config changes
* 1.1.0-beta0

* SDK version 1.1.0

* remove .yarn/versions/761d804c.yml
* Reduce logging from instrument

* FMT
* Fix provider wrapping order (#1436)

* Fix provider wrapping order

* Better RPC provider metrics

* Clean after cherry-pick

* Cleanup
* Gnosis deploy mainnet2 jan 26

* Fix rust

* PR review

* Set etheruem finality to 14 blocks

* Fix testnet verification artifacts

* fix artifact

* Fix key funding logic

* Deploy agents
* Config for testnet3

* Config for mainnet2

* Removed newlines from debug print

* Print full hashes for matching list debug

* Deploy and config

* Fixes from PR review

* Fix compiler error

* Deploy gnosis rc
* Deploy SDK 1.1.3

* Update submodule references

* Yarn.lock

* Readme updates

---------

Co-authored-by: Asa Oines <[email protected]>
* key funder fixes

* Minor tweaks

* Updated configs

* Update funding to deploy to rc

* Deploy

* Better error handling

* Don't include empty addresses and log errors

* Deploy kathy to rc

* Deploy key funder
* Add transaction gas limit for relayer

* Don't run if it exceeds the limit

* Allow skipping transaction gas limits for some chains

* fmt

* Update log

* Update configs

* Undo

* disable max gas on main chains for now so we can test in rc first
* Leverage storage refunds in pausable guard

* Update snapshot
* Use httpFallback for rc

* Update sha for rc

* Add commit dates

* Deploy RC

* Deploy to testnet3

* Use fallback for testnet3
* Better image tags

* build image

* try 2

* try 3

* try 4

* Fix deprecation

* Include ref name

* Revert "Include ref name"

This reverts commit c9c0e7e.

* Remove nightly tags
@yorhodes yorhodes requested a review from asaj February 9, 2023 16:28
@yorhodes yorhodes marked this pull request as draft March 27, 2023 19:02
@yorhodes yorhodes closed this Apr 7, 2023
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.

6 participants