Skip to content

Make a decent Python package - #6

Draft
valeriupredoi wants to merge 4 commits into
mainfrom
cleanup_package
Draft

Make a decent Python package#6
valeriupredoi wants to merge 4 commits into
mainfrom
cleanup_package

Conversation

@valeriupredoi

Copy link
Copy Markdown

Start making p5rem into a decent Python package, ready for deployment.

@valeriupredoi valeriupredoi added the enhancement New feature or request label Apr 16, 2026

@bnlawrence bnlawrence left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good. Only query. You've remove the wheel bit. Will this still happily work building the c-bits?

Comment thread pyproject.toml
@@ -1,16 +1,21 @@
[build-system]
requires = ["setuptools>=64", "wheel"]

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

@bnlawrence setuptools and pip build wheels so the fairly obsolete "wheel" system backend is not needed anymore; having said that, we really don't want to distribute wheels anyway

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

How do folks get the c-version for their platform? Do they have to build it on install, and what happens if they don't have a compiler?

@valeriupredoi

Copy link
Copy Markdown
Author

I still need to add a GHA test here...

@valeriupredoi

Copy link
Copy Markdown
Author

I still need to add a GHA test here...

let's get this one in @bnlawrence - it'll sit inside PyActiveStorage anyway so no point adding more testing bells and whistles to it until then - or do you reckon a GHA CI test would be useful in the meantime for development?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants