Skip to content
This repository was archived by the owner on Mar 20, 2024. It is now read-only.

Commit d2b0d20

Browse files
author
bors-servo
authored
Auto merge of #177 - Manishearth:try-fix, r=jdm
Use None, not False, for try_choose default value False gets stored as 0 fixes #176 I also ran `update table pull set try_choose=null where try_choose=0;` on the database to clean up things affected by this. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/homu/177) <!-- Reviewable:end -->
2 parents cbbfe97 + f786798 commit d2b0d20

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

homu/action.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -204,7 +204,7 @@ def review_approved(state, realtime, approver, username,
204204
if sha_cmp(sha, state.head_sha):
205205
state.approved_by = approver
206206
state.try_ = False
207-
state.try_choose = False
207+
state.try_choose = None
208208
state.set_status('')
209209

210210
state.save()

0 commit comments

Comments
 (0)