Merge lp:~nikwen/account-polld/fix-typo into lp:~ubuntu-push-hackers/account-polld/trunk

Proposed by Niklas Wenzel
Status: Approved
Approved by: Jonas G. Drange
Approved revision: 100
Proposed branch: lp:~nikwen/account-polld/fix-typo
Merge into: lp:~ubuntu-push-hackers/account-polld/trunk
Diff against target: 12 lines (+1/-1)
1 file modified
cmd/account-polld/account_manager.go (+1/-1)
To merge this branch: bzr merge lp:~nikwen/account-polld/fix-typo
Reviewer Review Type Date Requested Status
Jonas G. Drange (community) Approve
Sergio Schvezov Approve
Niklas Wenzel (community) Approve
Review via email: mp+238816@code.launchpad.net

Commit message

Fix typo

Description of the change

Fix typo

To post a comment you must log in.
Revision history for this message
Niklas Wenzel (nikwen) wrote :

Ok, please don't merge this. I'm working on some bug fixes as I explained here: https://lists.launchpad.net/ubuntu-phone/msg10265.html
As one of the fixes is closely related to the error handling in that line, I'll change it in that merge proposal. Changing it separately will just cause merge conflicts.

review: Disapprove
Revision history for this message
Niklas Wenzel (nikwen) wrote :

Since I was asked to split the following merge proposal this open for review again: https://code.launchpad.net/~nikwen/account-polld/error-handling-fixes/+merge/238875

review: Needs Resubmitting
Revision history for this message
Niklas Wenzel (nikwen) :
review: Approve
Revision history for this message
Niklas Wenzel (nikwen) wrote :

What's about this one?

Revision history for this message
Sergio Schvezov (sergiusens) :
review: Approve
Revision history for this message
Niklas Wenzel (nikwen) wrote :

Thanks for approving. :)

Revision history for this message
Jonas G. Drange (jonas-drange) wrote :

Let's merge it to automatic now, and put it into OTA9 or the next account-polld release.

review: Approve
Revision history for this message
Niklas Wenzel (nikwen) wrote :

Let's just get it in with the next release. This MP has been waiting that long already, the typo probably won't do as much harm as it requires effort to land this on its own. ;)

Thank you for approving though. :D

Unmerged revisions

100. By Niklas Wenzel

Fix typo

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'cmd/account-polld/account_manager.go'
--- cmd/account-polld/account_manager.go 2014-10-02 13:58:49 +0000
+++ cmd/account-polld/account_manager.go 2014-10-19 09:47:18 +0000
@@ -76,7 +76,7 @@
76 }76 }
7777
78 if a.penaltyCount > 0 {78 if a.penaltyCount > 0 {
79 log.Printf("Leaving poll for account %d as penaly count is %d", a.authData.AccountId, a.penaltyCount)79 log.Printf("Leaving poll for account %d as penalty count is %d", a.authData.AccountId, a.penaltyCount)
80 a.penaltyCount--80 a.penaltyCount--
81 return81 return
82 }82 }

Subscribers

People subscribed via source and target branches