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

fix: Update structure and packaging #18

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
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.

1 participant