-
Notifications
You must be signed in to change notification settings - Fork 15
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
feat(base_layer): add types to config #855
Conversation
Benchmark movements: |
Benchmark movements: |
3f5435f
to
84b3504
Compare
Benchmark movements: |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #855 +/- ##
=======================================
Coverage 74.29% 74.29%
=======================================
Files 358 358
Lines 36280 36288 +8
Branches 36280 36288 +8
=======================================
+ Hits 26953 26961 +8
- Misses 7188 7190 +2
+ Partials 2139 2137 -2 ☔ View full report in Codecov by Sentry. |
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.
Reviewed 4 of 4 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @dan-starkware)
84b3504
to
23fa49d
Compare
Benchmark movements: |
9710837
to
f0ce810
Compare
23fa49d
to
333cd90
Compare
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.
Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @dan-starkware)
333cd90
to
07ad05a
Compare
Benchmark movements: |
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.
Reviewed 1 of 1 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @dan-starkware)
07ad05a
to
d70e877
Compare
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.
Reviewed 1 of 1 files at r4, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @dan-starkware)
- implement the TODO for adding types to the Config, rather than working with raw strings. - using `Url` for url and **re-exporting** `alloy`'s `Address` type in order to not externalize this implementation detail type unnecessarily. - Remove 2 errors that are now no longer the responsibility of this module, the callsite will have to handle the casting into the correct types. commit-id:0a043060
d70e877
to
5637076
Compare
Benchmark movements: |
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.
Reviewed 1 of 1 files at r5, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @dan-starkware)
Benchmark movements: |
with raw strings.
Url
for url and re-exportingalloy
'sAddress
type in orderto not externalize this implementation detail type unnecessarily.
module, the callsite will have to handle the casting into the correct
types.
commit-id:0a043060
Stack:
This change is