ubuntu-archive-tools

Merge ubuntu-archive-tools:unsubscribe-team into ubuntu-archive-tools:main

  1. Git
  2. lp:ubuntu-archive-tools
  3. unsubscribe-team
  4. Merge into main
Proposed by Steve Langasek
Status: Merged
Merged at revision: cc651aede61fd253e7ae16b810ff46e9bb085763
Proposed branch: ubuntu-archive-tools:unsubscribe-team
Merge into: ubuntu-archive-tools:main
Diff against target: 0 lines
Reviewer Review Type Date Requested Status
Łukasz Zemczak Approve
Brian Murray Approve
Review via email: mp+410233@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Brian Murray (brian-murray) :
review: Approve
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Basically +1 on the changes, but maybe before merging you could fix a few stylistic bits per pep8:

$ pep8 unsubscribe-team
unsubscribe-team:12:1: E302 expected 2 blank lines, found 1
unsubscribe-team:28:1: E302 expected 2 blank lines, found 1
unsubscribe-team:72:1: E305 expected 2 blank lines after class or function definition, found 1

But yeah, either way, please feel free to merge!

review: Approve

Preview Diff

Empty

Subscribers

People subscribed via source and target branches