-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
PEP 691: Copyright #2593
PEP 691: Copyright #2593
Conversation
Hmm, I don't understand why that Lint job is failing, I think the |
I think it's unhappy about the trailing |
This reverts commit d1121cd.
Okay |
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.
(But don't merge until all the authors approve.)
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.
Thanks!
Yeah, so long as its in the same thread, no need to link to multiple individual messages (I did have it allow that for mailing list posts, though, since there may sometimes be a need for it and its harder to do by accident, whereas with Discourse the |
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.
Thanks, @dstufft ! And PR approvals with a message also indicating such seem to be a pretty solid strategy for documenting license acceptance, since they are relatively permanent (ish) and harder to mutate or "take back" compared to most other interactions.
We'll leave it to you to merge once @pradyunsg approves.
Adds the Post History for TodayTo merge this, we'll need permission from all the existing authors to copyright their work under the included license.