Skip to content

Update license, repository metadata #15

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

Merged
merged 1 commit into from
May 13, 2025

Conversation

patsier-cms
Copy link
Contributor

@patsier-cms patsier-cms commented May 9, 2025

Update license, repository metadata

Problem

Great work getting this shipped! I took a look at the PyPi page and noticed it wasn't handling the license identifier and there wasn't a link back to the repo.

Screenshot 2025-05-09 at 7 16 36 AM

Solution

I made a small change to update the license identifier to the SPDX format by adding a dash. I also added the repository field under tool.poetry so that should render in the "Unverified Details" section and allow people to get back to the repo more easily.

Quick note that technically this is deprecated and Poetry is moving toward following the PyPA specification more closely, but that's a much bigger change and I don't imagine a high priority.

Test Plan

Not really a clear way to test without publishing that I'm aware of unless there's a dry run option

@sachin-panayil
Copy link
Collaborator

thanks for this!

@sachin-panayil sachin-panayil merged commit fbd2237 into DSACMS:main May 13, 2025
1 check passed
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