Two minor points:
+ info = sys.exc_info() + getUtility(IErrorReportingUtility).raising(info)
Why not just say getUtility(IErrorReportingUtility).raising(sys.exc_info())?
+ # If the diffstat isn't real, we want to record an oops but continue.
This comment isn't very clear. Maybe:
# If the diff is formatted such that generating the diffstat fails, we want to record # an oops but continue.
?
« Back to merge proposal
Two minor points:
+ info = sys.exc_info() IErrorReporting Utility) .raising( info)
+ getUtility(
Why not just say getUtility( IErrorReporting Utility) .raising( sys.exc_ info()) ?
+ # If the diffstat isn't real, we want to record an oops but continue.
This comment isn't very clear. Maybe:
# If the diff is formatted such that generating the diffstat fails, we want to record
# an oops but continue.
?