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

Check md/rst in License field #196

Closed
henryiii opened this issue Oct 10, 2024 · 1 comment
Closed

Check md/rst in License field #196

henryiii opened this issue Oct 10, 2024 · 1 comment

Comments

@henryiii
Copy link
Collaborator

henryiii commented Oct 10, 2024

It looks like it might be possible to break reading by putting some types of content in License. We should either find a way to escape it, or provide a warning if it will not be readable. See python/cpython#119650.

Not as easy as I thought, as packaging can read these files that importlib.metadata can't (the PyPI description is fine).

@henryiii
Copy link
Collaborator Author

I'm not able to reproduce now.

# /// script
# dependencies = ["nox"]
# ///

import nox
import nox.__main__


@nox.session(python=["3.8", "3.9", "3.10", "3.11", "3.12", "3.13"], venv_backend="uv")
def check(session):
    session.install("iminuit")
    session.run("python", "-c", "import importlib.metadata; importlib.metadata.metadata('iminuit')['License']")

if __name__ == "__main__":
    nox.__main__.main()

works.

@henryiii henryiii closed this as not planned Won't fix, can't repro, duplicate, stale Oct 21, 2024
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

No branches or pull requests

1 participant