Code review comment for lp:~akretion-team/server-env-tools/70-scheduler-error-mailer
Revision history for this message
| Stefan Rijnhart (Opener) (stefan-opener) wrote | # |
review:
Needs Information
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
a60fb26
(Get the code!)
Looks neat! No nits to pick, but only what might be a stupid question about the todo in line 269: can't you put the name of the database in the context like you do with the job_exception?