etsy / opsweekly

On call alert classification and reporting
MIT License
761 stars 100 forks source link

Bitbucket weekly provider #62

Closed ninoabbate closed 8 years ago

ninoabbate commented 8 years ago

I have added a readme that describes how this provider must be configured.

lozzd commented 8 years ago

This looks good and I'll merge.. one thing I noticed though is you're using syntax in this (and the JIRA and Github) SQL like "SELECT foo FROM opsweekly.table_name" - there is config to allow users to use different database names (since many teams can host different instances on one SQL server or might not be able to choose the MySQL database name on some hosting providers). Can you remove the "opsweekly." part?

Thanks!

ninoabbate commented 8 years ago

Done in pull request #63