Skip to content

Tidy tools #1294

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

Merged
merged 16 commits into from
Jul 19, 2021
Merged

Tidy tools #1294

merged 16 commits into from
Jul 19, 2021

Conversation

Yobmod
Copy link
Contributor

@Yobmod Yobmod commented Jul 19, 2021

Tidy up some tooling:

  • Remove tox dependancy and tox.ini.

  • Rmv deepsource anf codeclimate files

  • Rmv mypy.ini and.coveragerc

  • Add pyproject.toml (i tried setup.cfg first, but github actions didnt find [pytest] or [tool:pytest] from it)

  • Add .flake8 (flake8 doesn't use pyproject.toml yet)

  • update pythonpackage.yaml to add py3.10.0-beta.4 and rmv un-needed installs

  • Update requirements-dev.txt for local use.

  • Update sphinx to 4.1.1 & rmv py2 code from conf.py (autodoc gave errors for some typehinted sugnitures, but only in py3.6, but fixed in newer phinx)

  • Add autodoc-typehints

Yobmod added 16 commits July 19, 2021 13:59
Add pyproject.toml with sections for pyest, mypy, coverage.py
Add .flake8 file - flake8 wont use pyproject.toml without a wrapper.
e.g. flakehell or flake9
Not used anymore
Not used anymore
rmv unicode prefixes - sphinx 4+ wont accept py2 code
Add comment and more local libs
Rmv unneeded installs and testing flags (will use the flage from the config files)
Add flags from pythonpackage.yaml
Add --force sugar flag
Update testing section
@Byron Byron merged commit acbd6ba into main Jul 19, 2021
@Byron
Copy link
Member

Byron commented Jul 19, 2021

Great work, thanks so much! I can't wait to see if this improves the sphinx docs built by readthedocs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants