{"action":"created","issue":{"url":"https://api.github.com/repos/Workiva/dart_dev/issues/134","repository_url":"https://api.github.com/repos/Workiva/dart_dev","labels_url":"https://api.github.com/repos/Workiva/dart_dev/issues/134/labels{/name}","comments_url":"https://api.github.com/repos/Workiva/dart_dev/issues/134/comments","events_url":"https://api.github.com/repos/Workiva/dart_dev/issues/134/events","html_url":"https://github.com/Workiva/dart_dev/pull/134","id":135943602,"number":134,"title":"Update logic around including unit tests by default","user":{"login":"jayudey-wf","id":4429043,"avatar_url":"https://avatars.githubusercontent.com/u/4429043?v=3","gravatar_id":"","url":"https://api.github.com/users/jayudey-wf","html_url":"https://github.com/jayudey-wf","followers_url":"https://api.github.com/users/jayudey-wf/followers","following_url":"https://api.github.com/users/jayudey-wf/following{/other_user}","gists_url":"https://api.github.com/users/jayudey-wf/gists{/gist_id}","starred_url":"https://api.github.com/users/jayudey-wf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jayudey-wf/subscriptions","organizations_url":"https://api.github.com/users/jayudey-wf/orgs","repos_url":"https://api.github.com/users/jayudey-wf/repos","events_url":"https://api.github.com/users/jayudey-wf/events{/privacy}","received_events_url":"https://api.github.com/users/jayudey-wf/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2016-02-24T03:35:11Z","updated_at":"2016-02-24T13:30:31Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Workiva/dart_dev/pulls/134","html_url":"https://github.com/Workiva/dart_dev/pull/134","diff_url":"https://github.com/Workiva/dart_dev/pull/134.diff","patch_url":"https://github.com/Workiva/dart_dev/pull/134.patch"},"body":"## Issue\r\n- With inclusion of https://github.com/Workiva/dart_dev/pull/131 the way unit tests were being included wasn't working as expected.\r\n\r\n[6:20 PM] Greg Littlefield: FYI Our builds started failing after dart_dev 1.1.0 was released: https://github.com/Workiva/web_skin_dart/pull/284. In case anyone runs into a similar issue.\r\n\r\n## Changes\r\n**Source:**\r\n- Updated the logic to see if the cli param was explicitly set to false\r\n- Updated the tests that runs default unit tests to fail if unit tests aren't being run by default\r\n\r\n**Tests:**\r\n- updated\r\n\r\n## Areas of Regression\r\n- the `test` task\r\n\r\n## Testing\r\n- verify that `test` task is working as expected\r\n\r\n## Code Review\r\n@trentgrover-wf\r\n@maxwellpeterson-wf\r\n@evanweible-wf\r\n@dustinlessard-wf"},"comment":{"url":"https://api.github.com/repos/Workiva/dart_dev/issues/comments/188255911","html_url":"https://github.com/Workiva/dart_dev/pull/134#issuecomment-188255911","issue_url":"https://api.github.com/repos/Workiva/dart_dev/issues/134","id":188255911,"user":{"login":"dustinlessard-wf","id":2326518,"avatar_url":"https://avatars.githubusercontent.com/u/2326518?v=3","gravatar_id":"","url":"https://api.github.com/users/dustinlessard-wf","html_url":"https://github.com/dustinlessard-wf","followers_url":"https://api.github.com/users/dustinlessard-wf/followers","following_url":"https://api.github.com/users/dustinlessard-wf/following{/other_user}","gists_url":"https://api.github.com/users/dustinlessard-wf/gists{/gist_id}","starred_url":"https://api.github.com/users/dustinlessard-wf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dustinlessard-wf/subscriptions","organizations_url":"https://api.github.com/users/dustinlessard-wf/orgs","repos_url":"https://api.github.com/users/dustinlessard-wf/repos","events_url":"https://api.github.com/users/dustinlessard-wf/events{/privacy}","received_events_url":"https://api.github.com/users/dustinlessard-wf/received_events","type":"User","site_admin":false},"created_at":"2016-02-24T13:30:31Z","updated_at":"2016-02-24T13:30:31Z","body":"+1"}}