Merge lp://staging/~tony-badwolf/quickly/quicklyloggingupdate into lp://staging/quickly

Proposed by Tony Byrne
Status: Needs review
Proposed branch: lp://staging/~tony-badwolf/quickly/quicklyloggingupdate
Merge into: lp://staging/quickly
Diff against target: 145 lines (+42/-17)
7 files modified
data/templates/ubuntu-application/create.py (+1/-1)
data/templates/ubuntu-application/project_root/python/__init__.py (+16/-3)
data/templates/ubuntu-application/project_root/python_lib/Builder.py (+1/-1)
data/templates/ubuntu-application/project_root/python_lib/PreferencesDialog.py (+1/-1)
data/templates/ubuntu-application/project_root/python_lib/Window.py (+1/-1)
data/templates/ubuntu-application/project_root/python_lib/helpers.py (+8/-10)
data/templates/ubuntu-application/run.py (+14/-0)
To merge this branch: bzr merge lp://staging/~tony-badwolf/quickly/quicklyloggingupdate
Reviewer Review Type Date Requested Status
Quickly Developers Pending
Review via email: mp+84413@code.staging.launchpad.net

Description of the change

documents logging usage within code comments
restores original logging command line usage -d debug
all modules use same logger because some lib modules report on glade signals and preferences mismatches
default to logging level debug while project runs within quickly

To post a comment you must log in.

Unmerged revisions

643. By Tony Byrne <email address hidden> <email address hidden>

A more sensible default
Turn on logging while project runs inside quickly

642. By Tony Byrne <email address hidden> <email address hidden>

restores original "API" for logging
default logging.ERROR
-v logging.WARNING
-vv logging.INFO
-vvv logging.DEBUG
-d logging.DEBUG
fixes https://bugs.launchpad.net/quickly/+bug/882842

documents logging usage in __init__.py
addresses https://answers.launchpad.net/quickly/+question/176340

641. By Tony Byrne <email address hidden> <email address hidden>

don't hide logging messages from python_name_lib

640. By Didier Roche-Tolomelli

Merge David's fix for fixing signal autoconnect does not work with
signal names which contain hypens (LP: #890318). Thanks!

639. By Michael Terry

fix tests to pass with latest platform and quickly changes

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches