Docstrings should start on the same line as the opening quotes (line 8 of your patch).
It would be great to have a test that ensures update_security_contact_subscriptions works as expected and doesn't subscribe the security contact for non-security issues but does for security issues. Otherwise, seems good.
Hi Bryce,
Welcome to Launchpad!
Docstrings should start on the same line as the opening quotes (line 8 of your patch).
It would be great to have a test that ensures update_ security_ contact_ subscriptions works as expected and doesn't subscribe the security contact for non-security issues but does for security issues. Otherwise, seems good.