Skip to content

fix: matching of the "f" cli option to the fast var#111

Merged
besendorf merged 1 commit into
mvt-project:mainfrom
Nxtmaster10:fix/cli-fast-option-shorthand
Jul 17, 2026
Merged

fix: matching of the "f" cli option to the fast var#111
besendorf merged 1 commit into
mvt-project:mainfrom
Nxtmaster10:fix/cli-fast-option-shorthand

Conversation

@Nxtmaster10

Copy link
Copy Markdown
Contributor

the current cli option f enables verbose output. This PR changes this to the proper fast var.

@besendorf

Copy link
Copy Markdown
Collaborator

Thanks. So theres definatly a mistake there but Im not sure that this is the right solution:

	flag.BoolVar(&fast, "fast", false, "Fast mode")
	flag.BoolVar(&verbose, "f", false, "Fast mode")
	flag.BoolVar(&fast, "f", false, "Fast mode")

maybe we use v for verbose and also call it verbose mode and f for fast mode?

@Nxtmaster10

Copy link
Copy Markdown
Contributor Author

Thanks. So theres definatly a mistake there but Im not sure that this is the right solution:

	flag.BoolVar(&fast, "fast", false, "Fast mode")
	flag.BoolVar(&verbose, "f", false, "Fast mode")
	flag.BoolVar(&fast, "f", false, "Fast mode")

maybe we use v for verbose and also call it verbose mode and f for fast mode?

Looking at the current code this seems right:

	flag.BoolVar(&verbose, "verbose", false, "Verbose mode")
	flag.BoolVar(&verbose, "v", false, "Verbose mode")
	flag.BoolVar(&fast, "fast", false, "Fast mode")
	flag.BoolVar(&fast, "f", false, "Fast mode")

@besendorf

Copy link
Copy Markdown
Collaborator

Ah yeah sorry youre right. Thanks!

@besendorf
besendorf merged commit ad7f4ef into mvt-project:main Jul 17, 2026
1 check passed
@Nxtmaster10
Nxtmaster10 deleted the fix/cli-fast-option-shorthand branch July 17, 2026 14:32
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