-
Notifications
You must be signed in to change notification settings - Fork 13
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
Replace Config
with explicit register types.
#17
Conversation
Do we need a higher MSRV? |
|
Uh, sorry but commit fc24b44 contains way too many fiddly changes for me to review timely. It will take me a while to go over it. A mitigation would be splitting into several commits, if that is something you would be willing to do. |
Sure, I'll try to split some of this out. |
80e40bb
to
f8eebaf
Compare
I extracted some of this out into #19. |
775d15f
to
85bfab3
Compare
10d7665
to
232f47d
Compare
@eldruin, I think this should be good to go now. Don't think I can split this further. |
232f47d
to
3e7c5f2
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.
Looks great, thank you for your work!
No description provided.