search

damogranlabs / classy_blocks

Python classes for easier creation of OpenFOAM's blockMesh dictionaries.
MIT License
126 stars 35 forks source link

Package info and settings cleanup #25

Closed schperplata closed 1 year ago

schperplata commented 1 year ago

This file can be removed. ATM, Travis is not used and CI is configured in GitHub Actions.

Also, new standards are simplifying and unifying some package info. I think setup.py and requirements-dev.txt can be merged to pyproject.toml, similar to example here.

Should I look into it?

FranzBangar commented 1 year ago

In a jiffy!

schperplata commented 1 year ago

Done. Inspect some typing issues in tests: https://github.com/damogranlabs/classy_blocks/tree/feature/pkg_files_merge

Other than that it is ready for merge. Notice that now there is no more setup.py and or requirements.txt. It is all merged to a pyproject.toml. That also changes how development requirements are specified and installed (improved IMO): just install package with [dev] extra tag, for example: python -m pip install -e .[dev].

Additionally, version is only in one place, that is pyproject.toml, not in __init__.py.

All package and tools related stuff for classy_block project is now in pyroject.toml, except CI configuration, which remains in tox.ini and github __build_ci.yml__. What a wonderful world we live in. 😃

schperplata commented 1 year ago

Done f7eb41e2e7b4b7e398b9e66845af200d497b8f6c