Repository navigation
Conversation
Move to the org-standard packaging and release tooling (public-engineering#506): static PEP 621 metadata, a locked uv environment with PEP 735 dependency groups, a src/ layout, and python-semantic-release with OIDC publishing so releases are driven by conventional commits instead of manual tags. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
These targets used to delegate to tox, which set DJANGO_SETTINGS_MODULE and installed the doc tools. Now that they run the tools directly, export the settings for pylint-django and include the doc group in dev so `uv run make quality` and `uv run make docs` work from the dev environment. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
25 tasks
GitHub master blob URLs and Transifex paths still pointed at the old top-level edx_django_utils/ directory and would break after merge. The requirements/private.* ignore entries are dropped since requirements/ no longer exists. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
farhan
marked this pull request as ready for review
October 30, 2026 06:26
Contributor
|
Since |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Important
PR implemented with the assistance of Claude Code. Refined and validated before being submitted for code review.
Modernize
edx-django-utilsPart of openedx/public-engineering#506
Summary
edx-django-utils.src/layout.setup.py/setup.cfgwithpyproject.toml(PEP 621 static metadata).uvwith PEP 735 dependency groups; commituv.lock..coveragercintopyproject.toml.astral-sh/setup-uv; SHA-pin all actions.python-semantic-release+release.yml(OIDC trusted publishing), replacing the tag-triggeredpypi-publish.yml.Removed/Updated
Deleted files:
setup.py,setup.cfg,requirements/,.coveragerc,.github/workflows/pypi-publish.ymlUpdated Makefile targets:
requirementsuv sync --locked --group devinstead of pip-syncupgradeedx_lint write_uv_constraints+uv lock --upgradeinstead of pip-compilequalityDJANGO_SETTINGS_MODULE=test_settingsexported (tox now callsmake quality);setup.pydropped from the file listsisortmake isort)docsmake docs);python setup.py bdist_wheel→python -m build --wheel; thedevgroup now includesdocso this works frommake requirementstestpython -Wd -m pytest $(PYTEST_ARGS)so tox keeps master's-Wdand{posargs}behaviorVersioning
setuptools-scmwithdynamic = ["version"]— master had a PyPI publish workflow;python-semantic-releasecontrols the version string at release time via git tags.Important Notes
[tool.edx_lint].uv_constraintsaddssetuptools<82: the existingdoc8<1.0.0pin importspkg_resources, which setuptools 82 removed.ci.ymlno longer has apush:trigger;release.ymlcalls it viaworkflow_callon pushes tomaster. Thetestsjob name and matrix are unchanged.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 quality,make test,make docs,python -m build). Repo-owner is encouraged to run the repo's feature tests before merging.🤖 Generated with Claude Code