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.

Sign up for GitHub

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

Jump to bottom

cmd: fix config and flags loading process #56

Merged
merged 1 commit into from
Sep 7, 2023
Merged

cmd: fix config and flags loading process #56

merged 1 commit into from
Sep 7, 2023

Conversation

fanyang89
Copy link
Contributor

Get value from viper, like viper.GetString()

Value priority:

  1. Return value from config file
  2. Return value from flag value, passed by command line
  3. Return value from flag default value

@fanyang89 fanyang89 added the enhancement New feature or request label Sep 7, 2023
@fanyang89 fanyang89 self-assigned this Sep 7, 2023
@fanyang89
cmd: fix config and flags loading process
69b60fc 
Get value from viper, like `viper.GetString()`

Value priority:

1. Return value from config file
2. Return value from flag value, passed by command line
3. Return value from flag default value
@1023280072 1023280072 merged commit 77d2302 into main Sep 7, 2023
1 check passed
@1023280072 1023280072 deleted the fix-cmd branch September 7, 2023 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@1023280072 1023280072 1023280072 approved these changes

Assignees

@fanyang89 fanyang89

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@fanyang89 @1023280072