-
Notifications
You must be signed in to change notification settings - Fork 218
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
port to rust-ini 0.16 #152
Conversation
Where does this merge commit suddenly come from? |
I needed to resolve merge conflicts because rustfmt was run on the repo. |
Would you please remove the commit from this branch? |
You mean I should rebase my original change on the current master instead? Ok, I can do that ... |
I rebased my changes and bumped to rust-ini 0.16 while I was at it. All INI related tests pass. |
Are you still interested in this? If yes, I created a maintenance fork (read here). |
Feel free to take my patch, but my interest in the config crate is no longer as big as it was when I submitted this PR. |
This is merged in config-rs-maint as of master a3f205dc628b59a32de813b3efefbaec4667d217. |
Hey, would you please rebase this to latest master? |
str
s now, not Strings, so useto_owned()
, notclone()
to get an owned value from them