Skip to content

Conversation

jeronimoalbi
Copy link
Member

Checking errors is not required because flag values are initially validated on execution while setting the string values.

Closes #1101

Checking errors is not required because flag values are initially
validated on execution while setting the string values.
Copy link

codecov bot commented Feb 19, 2024

Codecov Report

Attention: 46 lines in your changes are missing coverage. Please review.

Comparison is base (82ec261) 24.78% compared to head (9a1f21f) 24.91%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3976      +/-   ##
==========================================
+ Coverage   24.78%   24.91%   +0.12%     
==========================================
  Files         296      296              
  Lines       24915    24792     -123     
==========================================
  Hits         6176     6176              
+ Misses      18172    18049     -123     
  Partials      567      567              
Files Coverage Δ
ignite/cmd/chain_debug.go 0.00% <0.00%> (ø)
ignite/cmd/generate_composables.go 0.00% <0.00%> (ø)
ignite/cmd/generate_hooks.go 0.00% <0.00%> (ø)
ignite/cmd/generate_vuex.go 0.00% <0.00%> (ø)
ignite/cmd/scaffold_band.go 0.00% <0.00%> (ø)
ignite/cmd/scaffold_chain.go 0.00% <0.00%> (ø)
ignite/cmd/scaffold_map.go 0.00% <0.00%> (ø)
ignite/cmd/generate_typescript_client.go 0.00% <0.00%> (ø)
ignite/cmd/scaffold_package.go 0.00% <0.00%> (ø)
ignite/cmd/scaffold_query.go 0.00% <0.00%> (ø)
... and 3 more

Copy link
Member

@julienrbrt julienrbrt left a comment

Choose a reason for hiding this comment

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

#1101 (comment) TIL.
ACK.

@jeronimoalbi jeronimoalbi merged commit d0bf0e9 into main Feb 19, 2024
@jeronimoalbi jeronimoalbi deleted the chore/remove-flag-getter-error-check branch February 19, 2024 15:40
julienrbrt pushed a commit that referenced this pull request May 29, 2024
* chore: remove error check after command flag get

Checking errors is not required because flag values are initially
validated on execution while setting the string values.

* chore: update changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:ci CI/CD workflow and automated jobs. component:cmd component:configs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

cmd: remove error checks when reading flags
2 participants