Skip to content

Add additional context to --debug log messages#28

Merged
binaryaaron merged 3 commits into
masterfrom
moar-logging
Apr 13, 2018
Merged

Add additional context to --debug log messages#28
binaryaaron merged 3 commits into
masterfrom
moar-logging

Conversation

@jrmontag

Copy link
Copy Markdown
Contributor

In the main() method, in debug mode, we print out the argument
dictionaries a few times but without context for which one is which.
This commit adds an extra line of debug logging each time that indicates
from which point in the process the args are originating. Hopefully this
will expedite any arg parsing debugging.

I've also removed one line of unused code that created an intermediate
object for the argparse object.

In the `main()` method, in debug mode, we print out the argument
dictionaries a few times but without context for which one is which.
This commit adds an extra line of debug logging each time that indicates
from which point in the process the args are originating. Hopefully this
will expedite any arg parsing debugging.

I've also removed one line of unused code that created an intermediate
object for the argparse object.
@CLAassistant

CLAassistant commented Apr 13, 2018

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@binaryaaron

Copy link
Copy Markdown

Thanks @jrmontag! Would you mind merging 3a6068f (latest master) and bumping the version in setup.py?

@binaryaaron binaryaaron merged commit b19217f into master Apr 13, 2018
@binaryaaron binaryaaron deleted the moar-logging branch April 13, 2018 21:01
@binaryaaron

Copy link
Copy Markdown

Great, thank you! I'll send a new release to Pypi.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants