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(yargs): specify types for every CLI option #878

Merged

Conversation

webknjaz
Copy link
Contributor

@webknjaz webknjaz commented Sep 3, 2022

Fixes #876

(see the backstory/archeology/investigation 🕵️‍♂️ in that issue linked above)

@webknjaz webknjaz requested a review from a team as a code owner September 3, 2022 20:59
@webknjaz webknjaz force-pushed the bugfixes/876-yargs-option-types branch from bc22780 to eb42885 Compare September 3, 2022 21:01
@webknjaz webknjaz force-pushed the bugfixes/876-yargs-option-types branch from eb42885 to 607c8b4 Compare September 3, 2022 21:11
@codecov
Copy link

codecov bot commented Sep 3, 2022

Codecov Report

Merging #878 (607c8b4) into main (dae22df) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #878   +/-   ##
=======================================
  Coverage   92.14%   92.14%           
=======================================
  Files          35       35           
  Lines        1222     1222           
  Branches      249      249           
=======================================
  Hits         1126     1126           
  Misses         64       64           
  Partials       32       32           
Flag Coverage Δ
alpine 92.14% <ø> (ø)
alpine-proxy 92.14% <ø> (ø)
alpine-without-git 92.14% <ø> (ø)
linux 92.14% <ø> (ø)
linux-without-git 92.14% <ø> (ø)
macos 92.14% <ø> (ø)
macos-without-git 92.14% <ø> (ø)
windows 92.14% <ø> (ø)
windows-without-git 92.14% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/helpers/cli.ts 100.00% <ø> (ø)

@webknjaz
Copy link
Contributor Author

@drazisil-codecov @thomasrockhu-codecov @mitchell-codecov ☝️ this is an easy merge, but it fixes a serious bug with flags being corrupted on upload. Mind taking a look? ☝️

Copy link
Contributor

@drazisil-codecov drazisil-codecov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@drazisil-codecov drazisil-codecov merged commit 74b1e86 into codecov:main Sep 13, 2022
@webknjaz
Copy link
Contributor Author

@drazisil-codecov thanks! Do you think a new release with this could be expedited?

@drazisil-codecov
Copy link
Contributor

@drazisil-codecov Yep, just checking what else is pending so I can make it worth it :D

This was referenced Sep 13, 2022
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.

[BUG] Flags resembling floating point numbers get corrupted on upload
2 participants