Skip to content

fix: Update structure and packaging #18

New issue

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

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

Already on GitHub? # to your account

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

12rambau
Copy link
Member

@12rambau 12rambau commented Feb 23, 2023

WIP

I'm in the mood these day so I went on a crusade to update packaging of the extention I'm using in my documentation. I love sphinx-contrib email so I'm taking the time to make few improvement.

I hop you will be interested.

So far this contains:

  • drop use of cfg files in favor of only pyproject.toml
  • sanytise pre-commits (I think you are using some from another package and many things do were not required (notebooks, html etc) with the combo prettier/ruff/black everythin is covered
  • change structure of the files (the src directory is skiped anyway during the build so why keeping it ?
  • introduce nox for automated testing (mypy/linting)
  • create basic testing (that cover 84% of the code)
  • run a unit test on PR and push in github actions

@12rambau 12rambau mentioned this pull request Feb 23, 2023
@12rambau 12rambau marked this pull request as ready for review February 23, 2023 15:20
@12rambau 12rambau mentioned this pull request Feb 23, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant