Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Convert configuration to pyproject.toml, still using setuptools #917

Merged
merged 2 commits into from
Dec 5, 2023

Conversation

sirosen
Copy link
Member

@sirosen sirosen commented Dec 5, 2023

Convert from setup.cfg to pyproject.toml + .flake8.

This conversion was handled by starting with ini2toml[lite] output, then
reviewing and editing the results as appropriately (for example, ini2toml
does not correctly handle pylint config).

One unrelated change is included to tox.ini, as tox r -e lint was failing
for unrelated reasons (old version of pre-commit was being used).


📚 Documentation preview 📚: https://globus-sdk-python--917.org.readthedocs.build/en/917/

Convert to pyproject setuptools metadata.
The setup.py stub is left in place.

All tool configs are converted here as appropriate, including flake8
as `.flake8`.

Not all comments were preserved identically -- this was taken as an
opportunity to rewrite the comments in some cases.
Remove old things from the repo development history. The new list of
names reflects current development practices and files generated by
the repo.
@sirosen sirosen requested a review from aaschaer as a code owner December 5, 2023 20:35
@sirosen sirosen added the no-news-is-good-news This change does not require a news file label Dec 5, 2023
@sirosen sirosen merged commit 90c5afd into globus:main Dec 5, 2023
16 checks passed
@sirosen sirosen deleted the convert-to-pyproject branch December 5, 2023 21:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-news-is-good-news This change does not require a news file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants