Skip to content
This repository has been archived by the owner on Jun 27, 2018. It is now read-only.

db: remove violation table SQL and migrations #39

Merged
merged 1 commit into from Jan 23, 2017
Merged

db: remove violation table SQL and migrations #39

merged 1 commit into from Jan 23, 2017

Conversation

g-k
Copy link
Contributor

@g-k g-k commented Jan 20, 2017

Forgot to remove these with the switch to reading violations from the config file.

@g-k g-k requested review from jvehent and autrilla January 20, 2017 20:42
@g-k
Copy link
Contributor Author

g-k commented Jan 20, 2017

r? @autrilla @jvehent

jvehent
jvehent approved these changes Jan 20, 2017
View reviewed changes
Copy link

@jvehent jvehent left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r+
Side note: would be nice to run the monitoring lambda job against the app container in CI, as an integration test.

@g-k g-k mentioned this pull request Jan 23, 2017
Open
@g-k g-k merged commit 6e80eb4 into master Jan 23, 2017
@g-k g-k deleted the rm-old-violations-table branch January 23, 2017 17:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jvehent jvehent jvehent approved these changes

@autrilla autrilla Awaiting requested review from autrilla

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@g-k @jvehent