Skip to content

Move to the modern pyproject.toml settings file#56

Merged
mrtmm merged 1 commit intocleura:masterfrom
mrtmm:update-builds
Feb 16, 2026
Merged

Move to the modern pyproject.toml settings file#56
mrtmm merged 1 commit intocleura:masterfrom
mrtmm:update-builds

Conversation

@mrtmm
Copy link
Collaborator

@mrtmm mrtmm commented Feb 13, 2026

Using the pyproject.toml file is strongly recommended by the Python community and will help us move away from using the deprecated python setup.py sdist build command.
(https://packaging.python.org/en/latest/discussions/setup-py-deprecated/)

Add the pyproject.toml file and move the content onver from setup.py file.

Using the pyproject.toml file is strongly recommended by the
Python community and will help us move away from using the
deprecated `python setup.py sdist` build command.
(https://packaging.python.org/en/latest/discussions/setup-py-deprecated/)

Add the pyproject.toml file and move the content onver from setup.py
file.
@mrtmm mrtmm marked this pull request as ready for review February 13, 2026 13:19
@mrtmm mrtmm requested a review from fghaas February 13, 2026 13:19
Copy link

@fghaas fghaas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have just one question for the benefit of my own understanding. :)

[project.entry-points."xblock.v1"]
markdown = "markdown_xblock:MarkdownXBlock"

[tool.setuptools_scm]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it correct to assume that including this section causes the package to behave the same way it would with setup(use_scm_version=True)?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, exactly.

@mrtmm mrtmm merged commit 1f9e831 into cleura:master Feb 16, 2026
3 checks passed
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.

2 participants