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

(fix): add LICENSE #56

Merged
merged 3 commits into from
Nov 22, 2024
Merged

(fix): add LICENSE #56

merged 3 commits into from
Nov 22, 2024

Conversation

ilan-gold
Copy link
Owner

Fixes #55

@ilan-gold ilan-gold changed the title (fix): Create LICENSE (fix): add LICENSE Nov 22, 2024
@LDeakin
Copy link
Collaborator

LDeakin commented Nov 22, 2024

The conf.py currently has all of us under the copyright, do we want this consistent?

@ilan-gold
Copy link
Owner Author

ilan-gold commented Nov 22, 2024

The conf.py currently has all of us under the copyright

That's a good point. I only put one because the github form used the singular "author". I didn't check if we can put more than one in a license (probably?). I will look now

Update: https://opensource.stackexchange.com/questions/9774/how-to-fill-the-name-of-copyright-owner-in-the-license-if-there-are-multiple seems ok?

Copy link
Collaborator

@flying-sheep flying-sheep left a comment

Choose a reason for hiding this comment

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

should add it to the Python metadata

@flying-sheep flying-sheep merged commit a57b74c into main Nov 22, 2024
7 checks passed
@flying-sheep flying-sheep deleted the ig/license branch November 22, 2024 09:38
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.

Add license
3 participants