Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Consider using argparse module in the main app #10

Closed
palemieux opened this issue Sep 17, 2020 · 1 comment · Fixed by #18
Closed

Consider using argparse module in the main app #10

palemieux opened this issue Sep 17, 2020 · 1 comment · Fixed by #18
Assignees
@BobBrownConsulting
Labels
c-app for-review proposed resolution is ready for review

Comments

@palemieux
Copy link
Contributor

No description provided.

BobBrownConsulting added a commit that referenced this issue Sep 21, 2020
@BobBrownConsulting
Switching from getopt to argparse (#10)
6f69d5d
BobBrownConsulting added a commit that referenced this issue Sep 22, 2020
@BobBrownConsulting
Merging master to branch. Cleanup (#10)
f1ad59f
@palemieux palemieux added the for-review proposed resolution is ready for review label Sep 22, 2020
BobBrownConsulting added a commit that referenced this issue Sep 22, 2020
@BobBrownConsulting
Issues/10 switching from getopt to argparse (#18)
8296248 
* Adding main.py with argument parsing (input file, output file) - (#7)

* Refactoring. Adding more processing - (#7)

* Refactoring. Adding dummy writer for imsc, unit test, and hooking in main.py - (#7)

* Cleanup - (#7)

* Updates bringing in line with changes from master (#7)

* Fixing issues with unit tests and pylint (#7)

* Switching from getopt to argparse (#10)
@palemieux palemieux mentioned this issue Sep 22, 2020
Merged
@BobBrownConsulting
Copy link
Contributor

Closing

BobBrownConsulting added a commit that referenced this issue Sep 23, 2020
@BobBrownConsulting
Add unit test for CLI (#19)
e57eea4 
* Adding main.py with argument parsing (input file, output file) - (#7)

* Refactoring. Adding more processing - (#7)

* Refactoring. Adding dummy writer for imsc, unit test, and hooking in main.py - (#7)

* Cleanup - (#7)

* Updates bringing in line with changes from master (#7)

* Fixing issues with unit tests and pylint (#7)

* Switching from getopt to argparse (#10)

* Adding a unit test for main (#12)

* Merging master to branch. Cleanup. Updating unit test to pass in args properly (#12)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@BobBrownConsulting BobBrownConsulting

Labels
c-app for-review proposed resolution is ready for review
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Issues/10 switching from getopt to argparse sandflow/ttconv
2 participants
@palemieux @BobBrownConsulting