dmwm / dmwm/CRABClient

Try/except around optparse to catch sys.exit

Open
#4,283 0 comments 0 reactions 1 assignee Claimed by @emaszs View on GitHub
enhancement Low Priority
Dominant language
Python
Stars
18
Forks
38
PR merge metrics
No merged PRs in 30d

Description

Currently we let bad argument parsing throw SystemExit and then try to catch it far away. It's possible, however, other bits of code could be throwing the same exception and we wouldn't know.

When the client hits options.parse_args(), it should be wrapped in the tiniest try/except possible that then rethrows a custom exception there which can be better handled at higher layers

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.