-
Notifications
You must be signed in to change notification settings - Fork 191
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
remove SequencerConfig
usage
#2467
Comments
I will like to work on this |
hey @Jonatan-Chaverri, im still figuring out that part as i refactor stuff around. so not really thinking of opening it to external contributor for now. if you want to contribute, i'd suggest trying this one instead #2458 |
@tcoratger can i take up this issue |
@tcoratger want to work on this issue assign it to me |
@ScottyDavies @Sagar-Puniyani sorry guys, we're not accepting external contributors for this issue for now. |
the entire config management has been revamped here #2508 |
SequencerConfig
is marked as in the process of removal indojo/crates/katana/core/src/sequencer.rs
Lines 1 to 10 in 60c1575
There are still a few places in the code where this structure is used, for example here:
dojo/crates/dojo-test-utils/src/sequencer.rs
Line 38 in 60c1575
In order to fully deprecate this type, we need to remove uses like this.
The text was updated successfully, but these errors were encountered: