zavolanlab / htsinfer

Infer metadata for your downstream analysis straight from your RNA-seq data
Apache License 2.0
9 stars 22 forks source link

fix: add default to path_2_processed arg #137

Closed balajtimate closed 10 months ago

balajtimate commented 10 months ago

Description

Add None to path_2_processed: Optional[Path] as default, to prevent the validation error by Pydantic when initializing the Args class. This caused an issue during the publish docker image step of the CI.

Type of change

Checklist

Please carefully read these items and tick them off if the statements are true or do not apply.

If for some reason you are unable to tick off all boxes, please leave a comment explaining the issue you are facing so that we can work on it together.

codecov[bot] commented 10 months ago

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (58d7bbf) 99.80% compared to head (32e11f4) 99.80%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## dev #137 +/- ## ======================================= Coverage 99.80% 99.80% ======================================= Files 12 12 Lines 1039 1039 ======================================= Hits 1037 1037 Misses 2 2 ``` | [Files Changed](https://app.codecov.io/gh/zavolanlab/htsinfer/pull/137?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zavolanlab) | Coverage Δ | | |---|---|---| | [htsinfer/models.py](https://app.codecov.io/gh/zavolanlab/htsinfer/pull/137?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zavolanlab#diff-aHRzaW5mZXIvbW9kZWxzLnB5) | `100.00% <100.00%> (ø)` | |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.