Skip to content

fix: normalize invalid timeout parser errors#2867

Open
kuishou68 wants to merge 1 commit intosherlock-project:masterfrom
kuishou68:fix/2866-timeout-argparse
Open

fix: normalize invalid timeout parser errors#2867
kuishou68 wants to merge 1 commit intosherlock-project:masterfrom
kuishou68:fix/2866-timeout-argparse

Conversation

@kuishou68
Copy link
Copy Markdown

Closes #2866

Summary

  • wrap non-numeric timeout parsing failures in ArgumentTypeError
  • keep zero/negative timeout validation unchanged
  • add a regression test covering invalid timeout inputs

Testing

  • not run in this sandbox (no Python 3 runtime is available here)

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.

cli: invalid --timeout values raise a raw parser exception

1 participant