You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
PR #659 introduced a new shared argument --log-level which sets the level in which we'll output information with a logger object. This argument is now working, however it's not easy to access the LOG attribute that will be shared amongst all command classes. Currently, the way to access LOG goes as such:
This sets the logger object at the correct log level, but this does not actually share the same logger object with all command classes. We need to figure out a method to parse --log-level before all command classes are initialized, so they can all inherit the same logger object from GeoipsCommand.
Requested Update
Description
PR #659 introduced a new shared argument
--log-level
which sets the level in which we'll output information with a logger object. This argument is now working, however it's not easy to access theLOG
attribute that will be shared amongst all command classes. Currently, the way to access LOG goes as such:This sets the logger object at the correct log level, but this does not actually share the same logger object with all command classes. We need to figure out a method to parse
--log-level
before all command classes are initialized, so they can all inherit the same logger object fromGeoipsCommand
.Background and Motivation
This comes from this comment on PR #659.
Code to demonstrate issue
geoips.commandline.commandline_interface:GeoipsCLI:execute_command
The text was updated successfully, but these errors were encountered: