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: typing of version-modules #576

Merged
merged 3 commits into from
Jan 31, 2023
Merged

Fix: typing of version-modules #576

merged 3 commits into from
Jan 31, 2023

Conversation

Tacire
Copy link
Contributor

@Tacire Tacire commented Jan 27, 2023

What

Fixed typing errors
Tested using poetry run mypy pontos/version

Why

To prepare pontos for possible automated type-checkes in the future

Checklist

  • Tests

@Tacire Tacire requested a review from a team as a code owner January 27, 2023 15:12
@codecov
Copy link

codecov bot commented Jan 27, 2023

Codecov Report

Merging #576 (304741d) into main (59ca40d) will not change coverage.
The diff coverage is 85.71%.

@@           Coverage Diff           @@
##             main     #576   +/-   ##
=======================================
  Coverage   87.89%   87.89%           
=======================================
  Files          70       70           
  Lines        4751     4751           
  Branches      634      634           
=======================================
  Hits         4176     4176           
  Misses        412      412           
  Partials      163      163           
Impacted Files Coverage Δ
pontos/version/python.py 90.12% <66.66%> (ø)
pontos/version/cmake.py 90.72% <100.00%> (ø)
pontos/version/helper.py 65.62% <100.00%> (ø)

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

@bjoernricks bjoernricks merged commit a2e721f into main Jan 31, 2023
@bjoernricks bjoernricks deleted the Tacire/typing-version branch January 31, 2023 08:18
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.

2 participants