build: migrate to uv + pyproject.toml (PEP 621/735) + python-semantic-release - #253
Conversation
eb00a70 to
bca468d
Compare
a22bce9 to
c4c4cbd
Compare
008e1b6 to
307bdd8
Compare
008d221 to
e8ecbb0
Compare
|
@brian-smith-tcril I closed the older stale PR #237 and opened this. Salman has given 👍 on this PR, you can start doing its testing and review. @feanil review is appreciated on this important PR. |
|
Could you test this on |
|
I can test it later today |
|
@farhan can you please resolve the merge conflicts before we test? |
2d23a60 to
f677998
Compare
…-release Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
f677998 to
bca9dc9
Compare
|
@saraburns1 conflicts resolved |
|
Tests are running here: openedx/tutor-contrib-aspects#1343 |
bmtcril
left a comment
There was a problem hiding this comment.
Looks good, the tutor-contrib-aspects tests all passed 👍
|
@farhan We got an error on CI of merging this that prevented release from going out: I'll look more if I get time before you do. |
|
Problem was a race since CI was basically running twice at the same time. More details and proposed fix are here: #257 |
Important
PR implemented with the assistance of Claude Code. Refined and validated before being submitted for code review.
Modernize `platform-plugin-aspects`
Part of openedx/public-engineering#506
Summary
Removed/Updated
Deleted files: `setup.py`, `setup.cfg`, `requirements/`, `.coveragerc`, `.github/workflows/pypi-publish.yml`
Removed Makefile targets:
Updated Makefile targets:
Versioning
[Dynamic] `setuptools-scm` with `dynamic = ["version"]` — master had a PyPI publish workflow; `python-semantic-release` controls the version string at release time via git tags.
Important Notes
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`, `pytest`, `python -m build`). Repo-owner is encouraged to run the repo's feature tests before merging.
🤖 Generated with Claude Code