Skip to content

A flag with a non-zero default value will now show the default in the help#133

Merged
FollowTheProcess merged 1 commit intomainfrom
feat/default-non-zero-flags
Jan 27, 2025
Merged

A flag with a non-zero default value will now show the default in the help#133
FollowTheProcess merged 1 commit intomainfrom
feat/default-non-zero-flags

Conversation

@FollowTheProcess
Copy link
Owner

Summary

@FollowTheProcess FollowTheProcess added the feature New feature or request label Jan 27, 2025
@codecov
Copy link

codecov bot commented Jan 27, 2025

Codecov Report

Attention: Patch coverage is 92.59259% with 2 lines in your changes missing coverage. Please review.

Project coverage is 91.90%. Comparing base (1fcb29b) to head (8f738ce).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
internal/flag/flag.go 92.59% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #133      +/-   ##
==========================================
+ Coverage   91.88%   91.90%   +0.01%     
==========================================
  Files           6        6              
  Lines        1344     1371      +27     
==========================================
+ Hits         1235     1260      +25     
- Misses         77       79       +2     
  Partials       32       32              

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

@FollowTheProcess FollowTheProcess merged commit 1a133ac into main Jan 27, 2025
12 checks passed
@FollowTheProcess FollowTheProcess deleted the feat/default-non-zero-flags branch January 27, 2025 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant