-
Notifications
You must be signed in to change notification settings - Fork 14k
update mit-license and copyright #20919
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
|
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon. If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes. Please see CONTRIBUTING.md for more information. |
|
r? @brson |
|
We should probably fill this in as well :) https://github.com/rust-lang/rust/blob/master/LICENSE-APACHE#L189 |
|
Whoops didn't see that when I was reviewing the License either! |
|
@sfackler That has been suggested many times, and I think that is just a template for 'how to apply this to your own work', i.e. it is not supposed to be filled in. |
|
@bors: r+ 602a7d1 |
|
⌛ Testing commit 602a7d1 with merge 583c5c5... |
Only made 2 changes: 1) Update the year to 2015 in LICENSE-MIT 2) Update the year in COPYRIGHT No other changes were made.
|
All of these tests have passed, but @bors got a little confused, so merging manually. |
…y-flyimport fix: Consider all matches for flyimport even when searched with a qualifier
…y-flyimport fix: Consider all matches for flyimport even when searched with a qualifier
Only made 2 changes:
No other changes were made.