Skip to content

Commit

Permalink
Auto merge of barosl#138 - edunham:newflake8, r=jdm
Browse files Browse the repository at this point in the history
remove bare excepts for flake8

Bandaid for the bug which was hit in servo#133

<!-- 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/138)
<!-- Reviewable:end -->
  • Loading branch information
bors-servo committed Oct 26, 2017
2 parents 387c218 + 6e177cb commit f34e399
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions homu/main.py
Expand Up @@ -1303,7 +1303,7 @@ def fetch_mergeability(mergeable_que):

state.set_mergeable(mergeable, que=False)

except:
except Exception:
print('* Error while fetching mergeability')
traceback.print_exc()

Expand Down Expand Up @@ -1336,7 +1336,7 @@ def check_timeout(states, queue_handler):

queue_handler()

except:
except Exception:
print('* Error while checking timeout')
traceback.print_exc()

Expand Down
2 changes: 1 addition & 1 deletion homu/server.py
Expand Up @@ -695,7 +695,7 @@ def sync_repo(repo_label, g):
synchronize(repo_label, g.repo_cfgs[repo_label], g.logger, g.gh,
g.states, g.repos, g.db, g.mergeable_que,
g.my_username, g.repo_labels)
except:
except Exception:
print('* Error while synchronizing {}'.format(repo_label))
traceback.print_exc()
raise
Expand Down

0 comments on commit f34e399

Please sign in to comment.