{"comment":{"url":"https://api.github.com/repos/LandRegistry/dm-deed-api/comments/14819914","html_url":"https://github.com/LandRegistry/dm-deed-api/commit/388aef81a9a99e8f6040df775cceef9cdfab0a0d#commitcomment-14819914","id":14819914,"user":{"login":"tsangsimon","id":1902430,"avatar_url":"https://avatars.githubusercontent.com/u/1902430?v=3","gravatar_id":"","url":"https://api.github.com/users/tsangsimon","html_url":"https://github.com/tsangsimon","followers_url":"https://api.github.com/users/tsangsimon/followers","following_url":"https://api.github.com/users/tsangsimon/following{/other_user}","gists_url":"https://api.github.com/users/tsangsimon/gists{/gist_id}","starred_url":"https://api.github.com/users/tsangsimon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tsangsimon/subscriptions","organizations_url":"https://api.github.com/users/tsangsimon/orgs","repos_url":"https://api.github.com/users/tsangsimon/repos","events_url":"https://api.github.com/users/tsangsimon/events{/privacy}","received_events_url":"https://api.github.com/users/tsangsimon/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":"","commit_id":"388aef81a9a99e8f6040df775cceef9cdfab0a0d","created_at":"2015-12-07T12:51:11Z","updated_at":"2015-12-07T12:51:11Z","body":"wouldn't it be better to have the migration after the if ! $integration_tests_pass condition block? no point doing it if tests fail as a change will be implemented in a fail state. "}}