Add pre-commit + isort

[leonlan]

Closes #90.

• Add pre-commit to poetry
• Add pre-commit configuration with current dev dependencies
• Add isort as new dependency
• Remove linters/type checkers from poetry dev dependencies

[codecov]

Codecov Report

Merging #112 (f1bc543) into master (b42f503) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #112   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           28        28           
  Lines          664       664           
=========================================
  Hits           664       664           

Impacted Files	Coverage Δ
alns/stop/MaxRuntime.py	100.00% <ø> (ø)
alns/select/RouletteWheel.py	100.00% <100.00%> (ø)
alns/select/SegmentedRouletteWheel.py	100.00% <100.00%> (ø)
alns/stop/NoImprovement.py	100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[N-Wouda]

If we move all dev stuff into pre-commit, can we then get rid of the dev dependencies in pyproject.toml? I think so, if we update the CI to install and run the pre-commit thing as well?

[N-Wouda]

If we move all dev stuff into pre-commit, can we then get rid of the dev dependencies in pyproject.toml?

Then we can probably also get rid of a lot of the settings in pyproject.toml.

[leonlan]

If we move all dev stuff into pre-commit, can we then get rid of the dev dependencies in pyproject.toml? I think so, if we update the CI to install and run the pre-commit thing as well?

I read somewhere that testing is not recommended to be part of pre-commit. I guess that's mainly in very large codebases, which have much longer tests than we do have here. I think it's fine if we do add pytest to pre-commit.

So yes, we can put everything in pre-commit.

[N-Wouda]

I read somewhere that testing is not recommended to be part of pre-commit. I guess that's mainly in very large codebases, which have much longer tests than we do have here. I think it's fine if we do add pytest to pre-commit.

I meant just the linting and static type checker stuff. The test and coverage stuff can stay, I think.

[N-Wouda]

Much better!