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

add "License :: OSI Approved :: NASA Open Source Agreement (NOSA)" #167

Merged
merged 3 commits into from
Mar 3, 2024

Conversation

ebo
Copy link
Contributor

@ebo ebo commented Jan 22, 2024

Request to add a new NASA Open Source Agreement (NOSA) classifier:

 "License :: OSI Approved :: NASA Open Source Agreement (NOSA)"

According to various documentation[*], NOSA is an approved OSI open-source license, but is not contained in any of the online classifier lists.

Inclusion will help facilitate using NOSA in various python projects.

src/trove_classifiers/__init__.py Outdated Show resolved Hide resolved
@ebo
Copy link
Contributor Author

ebo commented Feb 22, 2024 via email

@tcaduser
Copy link
Contributor

Hi @ebo, it looks to me like @di is just requesting you to accept the proposed change. @di asked me to do the same thing for my proposed license addition.

@ebo
Copy link
Contributor Author

ebo commented Feb 22, 2024 via email

@ebo
Copy link
Contributor Author

ebo commented Mar 1, 2024

do you want me to make these changes and resubmit the pull request? I was confused on what you wanted me to do next.

@di
Copy link
Member

di commented Mar 3, 2024

@ebo If the change I made is OK with you, we can merge this.

@ebo
Copy link
Contributor Author

ebo commented Mar 3, 2024 via email

Copy link
Member

@di di left a comment

Choose a reason for hiding this comment

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

Thanks! Hope you feel better.

@di di merged commit 2fca3e3 into pypa:main Mar 3, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants