Skip to content

Conversation

@JPEWdev
Copy link
Collaborator

@JPEWdev JPEWdev commented Mar 6, 2025

There are projects wanting to use the Python bindings, and pulling them from github is annoying, particularly because it's not allowed to publish a downstream package that contains these bindings if they are pulled from GitHub.

As such, enable publishing to PyPi

@JPEWdev JPEWdev requested a review from goneall March 6, 2025 15:24
@bact
Copy link
Collaborator

bact commented Mar 6, 2025

Ahh. I think there's a bug in GitHub web interface. You can't have the last line of the suggestion to be the ending fence "```". You better edit this in your own text editor. I'm sorry @JPEWdev

@bact
Copy link
Collaborator

bact commented Mar 6, 2025

Line 19 supposed to have the closing "```" :(

@JPEWdev
Copy link
Collaborator Author

JPEWdev commented Mar 6, 2025

Weird

Copy link
Member

@goneall goneall left a comment

Choose a reason for hiding this comment

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

I may not be the best reviewer for Python publishing, but the changes look good to me.

@goneall
Copy link
Member

goneall commented Mar 6, 2025

@maxhbr - would you mind giving this a quick review

There are projects wanting to use the Python bindings, and pulling them
from github is annoying, particularly because it's not allowed to
publish a downstream package that contains these bindings if they are
pulled from GitHub.

As such, enable publishing to PyPi

Signed-off-by: Joshua Watt <JPEWhacker@gmail.com>
@JPEWdev JPEWdev merged commit 0b59dc8 into spdx:main Mar 24, 2025
4 checks 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.

3 participants