Quality of Life Improvement: Updated sca.py menu

bdorney commented 5 years ago

Description

I had structured the ArgumentParser in a manner that required the user to supply the cardName and ohMask to get the help menu for a subcommand.

This is obviously not ideal. I've resolved this issue.

Supersedes #45, this PR addresses issues #43 and #44.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

Motivation and Context

Fixing bad design.

How Has This Been Tested?

Change was trivial.

Screenshots (if appropriate):

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

jsturdy commented 5 years ago

bump @bdorney

bdorney commented 5 years ago

Thanks for reminder. Busy with connectivity testing routine and training QC8'ers. WIP still.

bdorney commented 5 years ago

@jsturdy @mexanick added a parent parser to reduce option duplication.

bdorney commented 5 years ago

@jsturdy parent parser was implemented.

bdorney commented 5 years ago

~~Short answer “no.”~~

~~I could make a pinned post in Mattermost referencing a new usage example and an email to the online SW egroup. Would this be suitable?~~

bdorney commented 5 years ago

@jsturdy I've notified the users; could you point me to the GEMDOC twiki page so I can also update the usage example there? I don't have my bookmarks...

cms-gem-daq-project / reg_utils