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

make GetFlagString() process the default string #127

Closed
wants to merge 1 commit into from

Conversation

klaernie
Copy link
Contributor

@klaernie klaernie commented Feb 8, 2024

make GetFlagString() process the default value, not Cobra

since the dateFormat / --date has a default set to dd-mm-yyy, the GetFlagString() function does not ever attempt reading the value from the config file.

Now every GetFlagString() invocation explicitly lists the default values, like GetFlagInt() does, hence one can set the date format in the config file.

Type:

  • Bug fix
  • New feature
  • Breaking change
  • Documentation

Camera:

  • GoPro
  • Insta360
  • DJI
  • Android
  • This PR adds a new camera

Component:

  • Core logic
  • Import
  • Merging
  • Firmware Update
  • Video Manipulation

Checklist before approval:

  • Linter pass
  • Build pass

since the dateFormat / --date has a default set to dd-mm-yyy, the
GetFlagString() function does not ever attempt reading the value from
the config file.

Now every GetFlagString() invocation explicitly lists the default
values, like GetFlagInt() does, hence one can override the date format
in the config file.
@KonradIT
Copy link
Owner

KonradIT commented Sep 4, 2024

Hi! Finally time to update mmt again.

Picked up your changes in this PR: #133

@KonradIT KonradIT closed this Sep 4, 2024
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.

2 participants