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

PEP 776: Address some of pfmoore's comments #4310

Merged
merged 8 commits into from
Mar 27, 2025

Conversation

hoodmane
Copy link
Contributor

@hoodmane hoodmane commented Mar 19, 2025

This attempts to address most of @pfmoore's comments here: https://discuss.python.org/t/pep-776-emscripten-support/84996/11

  • Clarify that we are speaking of platform tags.
  • List some examples of full tags that are Pyodide-compatible
  • Include code to generate compatible tag list
  • Reference PR to add this code to pypa/packaging and specify that it will be merged.
  • Add section with recommendations for package installers
  • Add section with recommendations for package indexes
  • Add section about dependency specifier markers
  • Add section on trove classifier
  • Add note that adding platform tags should have no backwards compatibility implications
  • Add link to Pyodide's documentation on building, testing, and ci for package maintainers

📚 Documentation preview 📚: https://pep-previews--4310.org.readthedocs.build/

This attempts to address most of pfmoore's comments here:
https://discuss.python.org/t/pep-776-emscripten-support/84996/11

* Clarify that we are speaking of _platform_ tags.
* List some examples of full tags that are Pyodide-compatible
* Include code to generate compatible tag list
* Reference PR to add this code to pypa/packaging and specify that it will be merged.
* Add section about dependency specifier markers
* Add section on trove classifier
* Add note that adding platform tags should have no backwards compatibility implications
* Add link to Pyodide's documentation on building, testing, and ci for package maintainers
@hoodmane hoodmane requested a review from ambv as a code owner March 19, 2025 14:52
@hoodmane hoodmane requested a review from pfmoore March 19, 2025 14:53
@hoodmane hoodmane changed the title Address some of pfmoore's comments PEP 776: Address some of pfmoore's comments Mar 19, 2025
@hoodmane
Copy link
Contributor Author

@hugovk can you merge this as well? I want to split the PEP as instructed by @pfmoore but it would be convenient to merge all open changes first.

Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
@hoodmane
Copy link
Contributor Author

Thanks @hugovk!

@hugovk hugovk merged commit baaaa7f into python:main Mar 27, 2025
5 checks passed
@hugovk
Copy link
Member

hugovk commented Mar 27, 2025

And thank you!

@hoodmane hoodmane deleted the some-pfmoore-comments branch March 27, 2025 13:46
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