Skip to content

Use cobra command for argument handling #11

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

Merged
merged 1 commit into from
Feb 20, 2023

Conversation

spenserblack
Copy link
Owner

While this doesn't add any options or arguments, it does set up the
command to recognize them. Specifically, that it shouldn't receive
any arguments or options.

This results in an error message when a user executes the command with
additional arguments or options, so that usage like
git lzc --version won't result in the executable surprisingly
generating a commit.

Fixes #10

@spenserblack spenserblack linked an issue Feb 20, 2023 that may be closed by this pull request
While this doesn't add any options or arguments, it does set up the
command to recognize them. Specifically, that it shouldn't receive
any arguments or options.

This results in an error message when a user executes the command with
additional arguments or options, so that usage like
`git lzc --version` won't result in the executable surprisingly
generating a commit.

Fixes #10
@spenserblack spenserblack force-pushed the bugfix/10/handle-arguments branch from 7992287 to 335378f Compare February 20, 2023 18:28
@codecov
Copy link

codecov bot commented Feb 20, 2023

Codecov Report

Merging #11 (335378f) into main (79cc397) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #11   +/-   ##
=======================================
  Coverage   77.53%   77.53%           
=======================================
  Files           5        5           
  Lines         138      138           
=======================================
  Hits          107      107           
  Misses         24       24           
  Partials        7        7           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@spenserblack spenserblack merged commit dc36075 into main Feb 20, 2023
@spenserblack spenserblack deleted the bugfix/10/handle-arguments branch February 20, 2023 18:29
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.

Handle arguments
1 participant