openapi-core
openapi-core copied to clipboard
Suggestion for project structure/setup improvements
I'd like to suggest an (in my opinion) improvement of the project structure/setup and the involved tooling to ease contributions and maintenance.
- Use Poetry for dependency management and packaging. Poetry has a number of advantages over plain pip including proper dependency resolution, built-in dependency locking, specification of dev-dependencies, a unified CLI for dependency and package management, virtual env management, etc. Poetry is used in some popular projects including, e.g., isort, diagrams, Rasa among others. I've been using Poetry since v0.12 and believe that it's the best tool available at this point for managing Python projects.
- Use code formatters: Black for most of the code and isort for imports. Those two are the de-facto standard code formatters for Python.
- Use Tox for test automation, e.g. for running the test suite against all officially supported Python versions (and if applicable against different versions of specific dependencies).
- Use pre-commit to run code formatters and linters before a commit to avoid erroneous commits.
- Optionally replace Flake8 by Pylint for more sophisticated linting. This is a matter of taste, I personally like Pylint a lot.
- Write a contribution guide on how to set up a development environment, how to run the test suite (against multiple Python versions), etc.
If done well, this setup has a single source of truth for all (dev-)dependencies and tool configurations for local development environments, IDEs, pre-commit hooks and CI jobs, integrates with common IDEs like VS Code out of the box (e.g. formatters can be applied when saving a file), and offers reproducible environments and builds for all contributors. I have numerous private projects set up this way, the DX is very smooth.
What do you think? I'd be happy to send a PR.
Hi @sisp
Most of your points I have in my plans.
- Use Poetry for dependency management and packaging.
I want to move to Poetry, but first I need to migrate to pyproject.toml
setup
- Use code formatters: Black for most of the code and isort for imports.
That's in high priority list also connected to pyproject.toml
- Use Tox for test automation
Would be nice to have it.
- Use pre-commit
Definitely want to use it
- Optionally replace Flake8 by Pylint
Would be nice to have it also.
- Write a contribution guide on how to set up a development environment
Good idea.
Thanks for your suggestions. I really appreciate it. Of course PRs are always welcome.
Cheers