{"action":"created","issue":{"url":"https://api.github.com/repos/oclint/oclint/issues/288","labels_url":"https://api.github.com/repos/oclint/oclint/issues/288/labels{/name}","comments_url":"https://api.github.com/repos/oclint/oclint/issues/288/comments","events_url":"https://api.github.com/repos/oclint/oclint/issues/288/events","html_url":"https://github.com/oclint/oclint/pull/288","id":128308615,"number":288,"title":"Improve oclint-rules test build times","user":{"login":"UnrealQuester","id":3252092,"avatar_url":"https://avatars.githubusercontent.com/u/3252092?v=3","gravatar_id":"","url":"https://api.github.com/users/UnrealQuester","html_url":"https://github.com/UnrealQuester","followers_url":"https://api.github.com/users/UnrealQuester/followers","following_url":"https://api.github.com/users/UnrealQuester/following{/other_user}","gists_url":"https://api.github.com/users/UnrealQuester/gists{/gist_id}","starred_url":"https://api.github.com/users/UnrealQuester/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/UnrealQuester/subscriptions","organizations_url":"https://api.github.com/users/UnrealQuester/orgs","repos_url":"https://api.github.com/users/UnrealQuester/repos","events_url":"https://api.github.com/users/UnrealQuester/events{/privacy}","received_events_url":"https://api.github.com/users/UnrealQuester/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-01-23T06:14:19Z","updated_at":"2016-01-23T17:04:59Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/oclint/oclint/pulls/288","html_url":"https://github.com/oclint/oclint/pull/288","diff_url":"https://github.com/oclint/oclint/pull/288.diff","patch_url":"https://github.com/oclint/oclint/pull/288.patch"},"body":"This PR merges the small test files into one executable per category and cleans up the includes for each file. This brings down the linux build time by 2 minutes and the osx build time by 4."},"comment":{"url":"https://api.github.com/repos/oclint/oclint/issues/comments/174200639","html_url":"https://github.com/oclint/oclint/pull/288#issuecomment-174200639","issue_url":"https://api.github.com/repos/oclint/oclint/issues/288","id":174200639,"user":{"login":"ryuichis","id":4386920,"avatar_url":"https://avatars.githubusercontent.com/u/4386920?v=3","gravatar_id":"","url":"https://api.github.com/users/ryuichis","html_url":"https://github.com/ryuichis","followers_url":"https://api.github.com/users/ryuichis/followers","following_url":"https://api.github.com/users/ryuichis/following{/other_user}","gists_url":"https://api.github.com/users/ryuichis/gists{/gist_id}","starred_url":"https://api.github.com/users/ryuichis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ryuichis/subscriptions","organizations_url":"https://api.github.com/users/ryuichis/orgs","repos_url":"https://api.github.com/users/ryuichis/repos","events_url":"https://api.github.com/users/ryuichis/events{/privacy}","received_events_url":"https://api.github.com/users/ryuichis/received_events","type":"User","site_admin":false},"created_at":"2016-01-23T17:04:58Z","updated_at":"2016-01-23T17:04:58Z","body":"Thanks @UnrealQuester again, very impressive. \r\n\r\nOne question, expect `CanaryTest`, all other test files include both `TestRuleOnCode.h` and `gtest/gtest.h` headers, then why not move `gtest` header inside `TestRuleOnCode`?"}}