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

Fill in the copyright info in the license template #1638

Closed
wants to merge 1 commit into from

Conversation

autarch
Copy link

@autarch autarch commented May 13, 2024

No description provided.

@mongodb-drivers-pr-bot mongodb-drivers-pr-bot bot added the priority-3-low Low Priority PR for Review label May 13, 2024
Copy link

API Change Report

No changes found!

@@ -186,7 +186,7 @@
same "printed page" as the copyright notice for easier
identification within third-party archives.

Copyright [yyyy] [name of copyright owner]
Copyright 2024 MongoDB, Inc.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We apply the explicit copyright on a per-file basis. It's unclear to me whether we intentionally leave this line blank. AFAICT no other drivers explicitly apply a copyright. @blink1073 do you have insight?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Correct. This is the appendix, and is meant to be written as-is. PyMongo and Java have the same appendix.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, I missed that this was the appendix.

@prestonvasquez
Copy link
Collaborator

@autarch Thank you for the PR, closing for now as it appears we are compliant per this thread.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority-3-low Low Priority PR for Review
Projects
None yet
3 participants