Skip to content
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

Restore copyright notice in main module #194

Merged
merged 1 commit into from
Aug 19, 2023
Merged

Restore copyright notice in main module #194

merged 1 commit into from
Aug 19, 2023

Conversation

jayaddison
Copy link
Contributor

With reference to this note from @pwdyson re: the removal of this notice:

Thanks James for drawing my attention to this. The copyright and my name should not be removed. Please put it back.

This reverts commit e640d3a.

@jayaddison
Copy link
Contributor Author

cc @jaraco - this is a small copyright notice adjustment, co-ordinated with Paul.

@jaraco
Copy link
Owner

jaraco commented Jul 27, 2023

Thanks for the ping. Let's first start with an issue where we can discuss the implications and tradeoffs.

@pwdyson
Copy link
Collaborator

pwdyson commented Jul 28, 2023

Please restore my name and copyright notice. Then we can discuss any changes.

@pwdyson

This comment was marked as off-topic.

@jayaddison

This comment was marked as off-topic.

@jayaddison
Copy link
Contributor Author

Quoting from https://www.linuxfoundation.org/blog/blog/copyright-notices-in-open-source-software-projects :

Don’t change someone else’s copyright notice without their permission

You should not change or remove someone else’s copyright notice unless they have expressly (in writing) permitted you to do so. This includes third parties’ notices in pre-existing code.

@jaraco
Copy link
Owner

jaraco commented Aug 19, 2023

See #197 (comment), where I lay out the plan.

@jaraco jaraco merged commit 8fb08de into jaraco:main Aug 19, 2023
@jayaddison jayaddison deleted the copyright-notice-restoration branch August 20, 2023 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants