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

Mypy Integration and Type Annotations Enhancement #1035

Merged
merged 4 commits into from
Mar 12, 2024
Merged

Conversation

Donaim
Copy link
Member

@Donaim Donaim commented Nov 22, 2023

This commit introduces Mypy type checking in the CI workflow to enhance code quality and robustness. Type annotations are added to consensus_aligner.py, report_amino.py, and project_config.py. A .mypy.ini file is added for Mypy configuration and relevant dependencies are included in requirements-dev.txt. The changes aim to catch type-related errors early and maintain clear function interfaces.

@Donaim Donaim requested a review from donkirkby November 22, 2023 19:52
Copy link

codecov bot commented Nov 22, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.07%. Comparing base (83cfe3c) to head (1b8b30b).

❗ Current head 1b8b30b differs from pull request most recent head 4b0fea6. Consider uploading reports for the commit 4b0fea6 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1035   +/-   ##
=======================================
  Coverage   88.07%   88.07%           
=======================================
  Files          29       29           
  Lines        7153     7155    +2     
=======================================
+ Hits         6300     6302    +2     
  Misses        853      853           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Donaim Donaim force-pushed the improve-type-checking branch from 459a895 to 46c2a49 Compare November 22, 2023 21:00
@Donaim Donaim marked this pull request as draft February 12, 2024 17:11
@Donaim Donaim force-pushed the improve-type-checking branch from 46c2a49 to 8d47f31 Compare March 11, 2024 19:16
@Donaim Donaim marked this pull request as ready for review March 11, 2024 19:19
@Donaim Donaim added this to the 7.16 milestone Mar 11, 2024
@Donaim Donaim force-pushed the improve-type-checking branch from f71c686 to bbdb716 Compare March 11, 2024 22:36
Donaim added 2 commits March 12, 2024 09:03
* consensus_aligner.py
* report_amino.py
* project_config.py

They are all imported by Contig stitcher.
@Donaim Donaim force-pushed the improve-type-checking branch from e386e19 to 3df103a Compare March 12, 2024 16:03
@Donaim Donaim force-pushed the improve-type-checking branch from 6c9542d to 1b8b30b Compare March 12, 2024 16:22
@Donaim Donaim merged commit 66c590f into master Mar 12, 2024
3 checks passed
@Donaim Donaim deleted the improve-type-checking branch March 12, 2024 18:19
@Donaim Donaim modified the milestones: 7.16, 7.17 May 8, 2024
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.

1 participant