-
Notifications
You must be signed in to change notification settings - Fork 12
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
👌 Refresh template plugin #88
Conversation
- Move to flit for PEP 621 compliant build - Remove setup.py, setup.json, and MANIFEST.in, check_version.py, .coveragerc - Move from yapf to black for code formatting - Add isort and pyupgrade pre-commit hooks - Move documentation from sphinx-rtd-theme to furo Note the setuptools `reentry_register` hook is no longer possible, although reentry will anyway be removed in aiida v2 (and I never found this to actually work).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks @chrisjsewell !
Move documentation theme from sphinx-rtd-theme to furo
Why furo?
Note the setuptools reentry_register hook is no longer possible, although reentry is anyway removed in AiiDA v2 (and I never found this to actually work?).
Indeed - e.g. this would anyhow not work when the package was distributed as a wheel, so no worries.
It's by me good friend and maintainer of pip; among others their documentation uses it: https://pip.pypa.io
Yeh, FYI I see why lots of the packages on aiida-registry didn't include a wheel lol: because the command in |
Updated according to aiidateam/aiida-plugin-cutter#88
* 👌 Update package Updated according to aiidateam/aiida-plugin-cutter#88 * Update pyproject.toml
setup.py
,setup.json
,MANIFEST.in
,check_version.py
, and.coveragerc
requires-python = ">=3.7"
yapf
toblack
for code formattingisort
andpyupgrade
pre-commit hookssphinx-rtd-theme
tofuro
tox
configuration (inpyproject.toml
)Note the setuptools
reentry_register
hook is no longer possible, although reentry is anyway removed in AiiDA v2 (and I never found this to actually work?).