[farhan]

Important

PR implemented with the assistance of Claude Code. Refined and validated before being submitted for code review.

Modernize enmerkar-underscore
Part of openedx/public-engineering#518

Summary

• enmerkar-underscore is published to PyPI: https://pypi.org/project/enmerkar-underscore/
• Replace setup.py/setup.cfg with pyproject.toml (PEP 621 static metadata)
• Switch from pip-compile to uv with PEP 735 dependency groups; commit uv.lock
• Retain flake8 as the quality linter, as on master
• Update CI to use astral-sh/setup-uv, run uv run tox, and SHA-pin all actions
• Add python-semantic-release + release.yml (OIDC trusted publishing), replacing the token-based publish_pypi.yml
• Version now derived from git tags via setuptools-scm; __version__ reads it from package metadata at runtime
• Drop Python 3.11 support; set requires-python = ">=3.12"

Removed/Updated

Deleted files: setup.py, setup.cfg, requirements/

Removed workflows: publish_pypi.yml (replaced by release.yml), upgrade-python-requirements.yml (its pip-compile upgrade flow is replaced by the upgrade Makefile target running uv lock --upgrade)

Updated Makefile targets:

Target	Change
requirements	Added — uv sync --group dev
upgrade	Now runs edx_lint write_uv_constraints + uv lock --upgrade (dropped all pip-compile calls)
lint	Now runs flake8 src tests (previously targeted a non-existent django-babel-underscore directory)
test	Now runs python -Wd -m pytest tests/ (dropped removed setup.py test)
coverage	Retargeted coverage source to src/enmerkar_underscore
docs	Simplified to make -C docs html (dropped sphinx-apidoc on a non-existent path)
release/dist	Now use python -m build + twine instead of setup.py sdist upload

Python 3.11 dropped

Removed Python 3.11 from the tox envlist, CI matrix, and classifiers; the target is 3.12.

Versioning

setuptools-scm with dynamic = ["version"] — the repo publishes to PyPI, so python-semantic-release controls the version string at release time via git tags. __version__ is exposed at runtime via importlib.metadata.

Important Notes

• Conventional commit format is now enforced on all future PRs to this repo via the existing commitlint.yml.
• The pytest-pep8, pytest-flakes, and python-coveralls test dependencies were dropped: pytest-pep8 is incompatible with modern pytest (uses removed hook arguments), and its pep8/pyflakes checks are already covered by flake8 in the quality env; python-coveralls is an unused, deprecated coveralls uploader.
• docs/conf.py contained invalid Python (import django-babel-underscore, version = django-babel-underscore.__version__) that would fail any docs build; it now reads the version via importlib.metadata. A docs tox environment is now part of the CI matrix.
• make lint previously targeted a non-existent directory (effectively a no-op); now that it lints src tests, two pre-existing E302 blank-line issues were fixed.
• Vendored third-party code under src/enmerkar_underscore/vendor/ is excluded from flake8.
• The package already used a src/ layout on master; no layout change was needed.
• Baseline tag v2.4.1 has been pushed so python-semantic-release can correctly identify the latest release.

Testing Notes

This PR has not been manually tested against the repo's own features. Testing relied on CI checks and local agent tooling (make requirements, make lint, make test, python -m build). Repo-owner is encouraged to run the repo's feature tests before merging.

---

🤖 Generated with Claude Code

[feanil]

One last thing and then this is good to go.