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

Revert custom errors to require statements #59

Merged
merged 24 commits into from
Oct 19, 2023
Merged

Revert custom errors to require statements #59

Show file tree
Hide file tree
Changes from 22 commits
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
c8ec8ef
revert custom errors and remove redundant teleporter sender checks
Oct 13, 2023
e47a0a3
update to use error strings in require
Oct 16, 2023
f31b07f
add reason string lint and lint script
Oct 16, 2023
5590b59
update erc20Bridge to use require statements
Oct 16, 2023
6c2a6cd
update cross chain apps to use require statements and abi bindings
Oct 16, 2023
a47b9e6
cleanup error messages for require
Oct 16, 2023
3e5d6c1
update abi bindings
Oct 16, 2023
ebc937f
update error messages
Oct 17, 2023
1f1e10c
Merge remote-tracking branch 'origin/main' into revert-custom-errors
Oct 18, 2023
8f76809
additional custom error removals
Oct 18, 2023
1821c9e
remove additional custom errors and update bindings
Oct 18, 2023
91c7412
Merge remote-tracking branch 'origin/main' into revert-custom-errors
Oct 18, 2023
07e380e
change to >= check
Oct 18, 2023
491900d
change check to use > 0
Oct 18, 2023
51f3da9
Update contracts/src/Teleporter/TeleporterMessenger.sol
Oct 18, 2023
b7c17ab
Update contracts/src/CrossChainApplications/ERC20Bridge/ERC20Bridge.sol
Oct 18, 2023
5ba1868
Update contracts/src/CrossChainApplications/ERC20Bridge/ERC20Bridge.sol
Oct 18, 2023
faa2837
Update contracts/src/CrossChainApplications/ERC20Bridge/ERC20Bridge.sol
Oct 18, 2023
e2b68a1
update lint command to match ci
Oct 18, 2023
7855a5b
revert change for >= check to prevent bridging with no amount
Oct 18, 2023
75a9b0e
Merge remote-tracking branch 'origin/main' into revert-custom-errors
Oct 18, 2023
4104849
Merge remote-tracking branch 'origin/main' into revert-custom-errors
Oct 18, 2023
c87cc97
rename format error message functions and use lint script for ci
Oct 19, 2023
661e408
install global instead of local because using as command line
Oct 19, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 6 additions & 2 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,12 @@ The Teleporter protocol, on the other hand, is implemented at the smart contract

- [Overview](#overview)
- [Setup](#setup)
- [Docker Setup](#docker-setup)
- [General Setup](#general-setup)
- [Structure](#structure)
- [Building and Running](#building-and-running)
- [Build + Run + Test](#build--run--test)
- [Run tests on Fuji Testnet](#run-tests-on-fuji-testnet)
- [E2E tests](#e2e-tests)
- [Docs](#docs)
- [Resources](#resources)

Expand Down Expand Up @@ -70,7 +74,7 @@ The above steps are sufficient to run the included integration tests inside Dock
- `scripts/fuji/example-workflows/` includes example workflows that send transactions to interact with Teleporter contracts on Fuji subnets.
- `docker/` includes containerized setup for running a local setup of Teleporter, as well as a script to run each of the integration tests against the local network.

## Building and Running
## Build + Run + Test

- Get all submodules: `git submodule update --init --recursive`
- Install Docker as described in the setup section of the README in the root of this repository.
Expand Down
2 changes: 1 addition & 1 deletion abi-bindings/go/CrossChainApplications/ERC20Bridge/BridgeToken/BridgeToken.go
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading