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

Use pyproject.toml instead of old setup.py #616

Closed
wants to merge 2 commits into from

Conversation

leocardao
Copy link
Contributor

@leocardao leocardao commented Aug 1, 2023

Remove src/e3/init.py to use:
PEP 420 – Implicit Namespace Packages
See: https://peps.python.org/pep-0420/

Update tox.ini to run into an isolated build.

Update mypy.ini and .pre-commit-config.yaml to work with the changes.

@leocardao leocardao force-pushed the mr/cardao/pyproject branch 3 times, most recently from 358a56a to f079f7b Compare August 1, 2023 15:19
@leocardao leocardao self-assigned this Aug 1, 2023
@leocardao leocardao force-pushed the mr/cardao/pyproject branch 3 times, most recently from c9983d8 to 02a6ce1 Compare August 1, 2023 15:46
@leocardao leocardao marked this pull request as draft August 1, 2023 15:47
- Remove src/e3/__init__.py to use:
    PEP 420 – Implicit Namespace Packages
    See: https://peps.python.org/pep-0420/

- Update tox.ini to run into an isolated build.
- Update mypy.ini and .pre-commit-config.yaml to work with the changes.
- Remove pkg_ressources usage.
@leocardao leocardao force-pushed the mr/cardao/pyproject branch 5 times, most recently from cd0cf77 to 5423eb2 Compare August 3, 2023 16:22
@grouigrokon
Copy link
Contributor

As long as the checks are not ok, I do not think I will spend time reviewing :)

@leocardao leocardao closed this Mar 21, 2024
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