-
Notifications
You must be signed in to change notification settings - Fork 65
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
chore: toml fix #665
chore: toml fix #665
Conversation
Codecov Report
@@ Coverage Diff @@
## main #665 +/- ##
==========================================
+ Coverage 60.73% 61.02% +0.28%
==========================================
Files 26 28 +2
Lines 5629 5616 -13
==========================================
+ Hits 3419 3427 +8
+ Misses 2210 2189 -21
... and 2 files with indirect coverage changes 📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today! |
So this throws a foundry error if we put it there. Not sure where we should put this |
Maybe move back to the arbiter.toml and we can put more higher level configuration there |
bin/bind/test.rs
Outdated
@@ -0,0 +1,108 @@ | |||
#[allow(unused_imports)] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rename this file tests from test
Closes #646
This also takes the fork tests out of a parallel loop so they will take load off the CI and fail less frequently