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

pre-commit-config: Invoke mypy for static type checking #160

Merged
merged 4 commits into from Apr 4, 2022

Conversation

MarijnS95
Copy link
Contributor

This check is run on the CI but was accidentally forgotten from pre-commit-config, whereas it'd be nice to have it included in poetry run pre-commit run --all-files :)

@codecov
Copy link

codecov bot commented Mar 26, 2022

Codecov Report

Merging #160 (1638938) into main (1bcaff8) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #160   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            4         4           
  Lines          182       182           
  Branches        38        38           
=========================================
  Hits           182       182           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1bcaff8...1638938. Read the comment docs.

This check is run on the CI but was accidentally forgotten from
pre-commit-config, whereas it'd be nice to have it included in `poetry
run pre-commit run --all-files` :)
@klaasnicolaas klaasnicolaas added the enhancement Enhancement of the code, not introducing new features. label Mar 30, 2022
@klaasnicolaas klaasnicolaas merged commit 4754330 into klaasnicolaas:main Apr 4, 2022
@MarijnS95 MarijnS95 deleted the pre-commit-mypy branch April 4, 2022 11:39
@github-actions github-actions bot locked and limited conversation to collaborators May 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Enhancement of the code, not introducing new features.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants