-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Fix a serialization error on publish #4085
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
Conversation
r? @matklad |
r? @brson (rust_highfive has picked a reviewer for you, use r? to override) |
@bors r+ @alexcrichton You've said you've run the original code in the rewriting PR on tons of crates. If you have an infra for that left somewhere, it might be useful to do the same again, with the new implementation. |
CI failure seems legit. |
@bors r- |
In TOML we have to emit all keys before we emit all sub-tables, so to handle that for dependencies just transform everything to an elaborated version. Closes rust-lang#4081
📌 Commit d1e70e4 has been approved by |
Fix a serialization error on publish In TOML we have to emit all keys before we emit all sub-tables, so to handle that for dependencies just transform everything to an elaborated version. Closes #4081
☀️ Test successful - status-appveyor, status-travis |
In TOML we have to emit all keys before we emit all sub-tables, so to handle
that for dependencies just transform everything to an elaborated version.
Closes #4081