Skip to content

fix: remove PEP 639 conflicting OSI license classifier - #6

Open
lukisch wants to merge 1 commit into
mainfrom
fix/license-classifier-conflict
Open

lukisch wants to merge 1 commit into
mainfrom
fix/license-classifier-conflict

Conversation

@lukisch

@lukisch lukisch commented Sep 26, 2026

Copy link
Copy Markdown
Contributor

Fixes T-20260926-656511331.

Current setuptools rejects the PEP 639 license = "MIT" field together with an OSI license classifier. This change removes only License :: OSI Approved :: MIT License from [project].classifiers and deletes its stale assertion in tests/test_metadata.py; license = "MIT" remains unchanged (this project has no license-files field).

Evidence from a fresh venv:

  • pip install -e . — exit 0; log: Successfully built llm-note and Successfully installed llm-note-1.0.4.
  • python -c "import llm_note" — exit 0; output: import ok: llm_note.
  • pytest -q tests/test_metadata.py — 11 passed in 1.76s.

…(T-20260926-656511331)

Cause: current setuptools rejects License classifiers alongside PEP 639 project.license. Removed only the MIT OSI classifier and its stale metadata-test expectation; kept license="MIT" (no license-files field exists). Fresh pip install -e .: Exit 0, Successfully installed llm-note-1.0.4. Fresh import llm_note: Exit 0. pytest -q tests/test_metadata.py: 11 passed in 1.76s.
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.

1 participant