Merge lp://staging/~mvo/software-center/fix-new-pep8 into lp://staging/software-center

Proposed by Michael Vogt
Status: Merged
Merged at revision: 3100
Proposed branch: lp://staging/~mvo/software-center/fix-new-pep8
Merge into: lp://staging/software-center
Diff against target: 433 lines (+43/-44)
19 files modified
softwarecenter/backend/installbackend_impl/aptd.py (+3/-3)
softwarecenter/backend/installbackend_impl/packagekitd.py (+2/-2)
softwarecenter/backend/launchpad.py (+1/-1)
softwarecenter/backend/weblive.py (+1/-1)
softwarecenter/backend/weblive_pristine.py (+6/-6)
softwarecenter/db/appfilter.py (+2/-2)
softwarecenter/db/categories.py (+1/-1)
softwarecenter/db/pkginfo_impl/aptcache.py (+7/-7)
softwarecenter/ui/gtk3/app.py (+3/-3)
softwarecenter/ui/gtk3/panes/historypane.py (+1/-1)
softwarecenter/ui/gtk3/panes/pendingpane.py (+1/-1)
softwarecenter/ui/gtk3/utils.py (+2/-3)
softwarecenter/ui/gtk3/views/appdetailsview.py (+2/-2)
softwarecenter/ui/gtk3/views/purchaseview.py (+1/-1)
softwarecenter/ui/gtk3/widgets/apptreeview.py (+1/-1)
softwarecenter/ui/gtk3/widgets/recommendations.py (+2/-2)
softwarecenter/ui/gtk3/widgets/reviews.py (+2/-2)
softwarecenter/ui/gtk3/widgets/weblivedialog.py (+1/-1)
softwarecenter/utils.py (+4/-4)
To merge this branch: bzr merge lp://staging/~mvo/software-center/fix-new-pep8
Reviewer Review Type Date Requested Status
Gary Lasker (community) Approve
Review via email: mp+119695@code.staging.launchpad.net

Description of the change

This branch fixes the pep8 issues that the quantal pep8 branch complains about.

To post a comment you must log in.
3080. By Michael Vogt

softwarecenter/ui/gtk3/widgets/apptreeview.py: fix a leftover pep8 issue

3081. By Michael Vogt

merged from trunk

3082. By Michael Vogt

fix some more pep8 issues from recent merges

Revision history for this message
Gary Lasker (gary-lasker) wrote :

This is very nice, thanks a lot for this, mvo!

Oh, I found that there's an issue that doesn't allow this branch to run on Precise (a crash in the weblive code where it has been updated to use isinstance). This has a very simple fix so I will make a branch for that and propose it for merging.

Thanks again!

review: Approve

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