Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Error handling with incorrect argument #45

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

ManishaKumari295
Copy link
Contributor

@ManishaKumari295 ManishaKumari295 commented Mar 26, 2024

Closed #37

Description

The incorrect argument when passed in --state-dir flag must throw up a valid error .

Change in behavior

The incorrect argument passed in --state-dir flag will throw up only the error causing the command to command and no --help flags will be shown.

Added

--

Changed

Since the agrument is passed to --stae-dir ,no suggessted argument list from help should be displayed .Only the error regarding wrong argument will be shown in command line.

Fixed

A better error handling is dealt here.

Change verification
Screenshot of the error handling is attached .
image

@ManishaKumari295 ManishaKumari295 self-assigned this Mar 26, 2024
@ManishaKumari295 ManishaKumari295 marked this pull request as ready for review May 23, 2024 09:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant