-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Fix license in Cargo.toml files #4616
Conversation
@bors r+ |
📌 Commit d743a89 has been approved by |
Fix license in Cargo.toml files Missed in #3269 changelog: none r? @Manishearth
💔 Test failed - status-appveyor |
Right, forgot the CI is broken. |
Huh, why's it broken? Do we have an issue for this? |
There is no issue opened but on every PR (since today?) fails with:
To me it looks like |
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.
It is recommended to use OR operator instead: https://doc.rust-lang.org/cargo/reference/manifest.html#package-metadata
I just copied it from topmost Line 14 in 737f0a6
I'll update it later or tomorrow. |
Co-Authored-By: lzutao <taolzu@gmail.com>
Ready for review. |
@bors r+ |
📌 Commit 4f1d907 has been approved by |
@Manishearth it won't work until #4628 is merged. I think Clippy could do |
@bors r- |
Let's merge it before Clippy breaks again. |
@bors r+ |
📌 Commit 4f1d907 has been approved by |
Fix license in Cargo.toml files Missed in #3269 changelog: none r? @Manishearth
☀️ Test successful - checks-travis, status-appveyor |
Missed in #3269
changelog: none
r? @Manishearth