Merge lp:~jml/launchpad/xxx-cleanup into lp:launchpad
Status: | Merged |
---|---|
Approved by: | Jonathan Lange |
Approved revision: | no longer in the source branch. |
Merged at revision: | 13296 |
Proposed branch: | lp:~jml/launchpad/xxx-cleanup |
Merge into: | lp:launchpad |
Diff against target: |
580 lines (+128/-123) 24 files modified
lib/canonical/launchpad/daemons/tachandler.py (+2/-3) lib/canonical/launchpad/doc/badges.txt (+1/-1) lib/canonical/launchpad/doc/displaying-numbers.txt (+1/-1) lib/canonical/launchpad/doc/menus.txt (+1/-1) lib/canonical/launchpad/doc/presenting-lengths-of-time.txt (+1/-1) lib/canonical/launchpad/ftests/__init__.py (+18/-4) lib/canonical/lazr/doc/menus.txt (+1/-1) lib/devscripts/autoland.py (+0/-4) lib/lp/app/browser/stringformatter.py (+72/-1) lib/lp/app/browser/tests/test_stringformatter.py (+10/-3) lib/lp/app/doc/displaying-dates.txt (+1/-1) lib/lp/app/doc/displaying-paragraphs-of-text.txt (+1/-1) lib/lp/app/doc/tales-email-formatting.txt (+1/-1) lib/lp/app/doc/tales-macro.txt (+1/-1) lib/lp/app/doc/tales.txt (+1/-1) lib/lp/bugs/doc/displaying-bugs-and-tasks.txt (+1/-1) lib/lp/code/model/revision.py (+1/-1) lib/lp/registry/browser/distributionsourcepackage.py (+4/-2) lib/lp/registry/browser/tests/milestone-views.txt (+1/-1) lib/lp/registry/doc/sourcepackage.txt (+2/-13) lib/lp/registry/doc/team-nav-menus.txt (+1/-1) lib/lp/scripts/utilities/warninghandler.py (+1/-3) lib/lp/soyuz/browser/sourcepackagerelease.py (+5/-74) lib/lp/testing/__init__.py (+0/-2) |
To merge this branch: | bzr merge lp:~jml/launchpad/xxx-cleanup |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
j.c.sackett (community) | Approve | ||
Review via email: mp+65663@code.launchpad.net |
Commit message
[r=jcsackett]
Description of the change
This branch cleans up a few of my XXX comments from over the years.
* lib/lp/
* canonical.
* Wrap a syscall in tachandler with until_no_eintr, so that it's robust
* Put an __all__ in c.l.ftests to avoid pyflakes lint
* Removed a XXX comment that was relevant only when the bug was open
* Got rid of a duplicate call to get_authors()
* Passed through 'line' in our warninghandler, now that we aren't on Python 2.5
Happy to answer questions. Sorry for the grab bag approach, but it seemed easier for all concerned. Mostly me though.
Looks good.