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

Change copyright notice to specify all contributors #47

Merged
merged 2 commits into from
Jun 3, 2024

Conversation

joeloskarsson
Copy link
Collaborator

Motivation

As more people are now contributing to the code the copyright does not just belong to me and Tomas. To avoid having to update this with the name of every person contributing, I suggest to take some inspiration from e.g. https://github.com/pyg-team/pytorch_geometric/blob/master/LICENSE and https://github.com/numpy/numpy/blob/main/LICENSE.txt and use a general formulation "Neural-LAM contributors".

Description of change

Change the copyright notice in the MIT license to "Neural-LAM Contributors". The year can stay 2023, as that is the first year the work (the code) was published. As me and Tomas are included under "Neural-LAM Contributors" this is strictly expanding the number of copyright holders.

@joeloskarsson joeloskarsson self-assigned this Jun 2, 2024
@leifdenby
Copy link
Member

As more people are now contributing to the code the copyright does not just belong to me and Tomas. To avoid having to update this with the name of every person contributing, I suggest to take some inspiration from e.g. https://github.com/pyg-team/pytorch_geometric/blob/master/LICENSE and https://github.com/numpy/numpy/blob/main/LICENSE.txt and use a general formulation "Neural-LAM contributors".

Sounds great to me! Thanks for suggesting this :)

@joeloskarsson joeloskarsson merged commit e5400bb into main Jun 3, 2024
8 checks passed
@joeloskarsson joeloskarsson deleted the license_name_update branch June 3, 2024 12:39
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.

2 participants