{"push_id":601771118,"size":1,"distinct_size":1,"ref":"refs/heads/master","head":"8c4190ca956a83f3286bc0d104041e94d53eaa59","before":"3208b06ce832c884bdb3b95cebeefa5bd55067ba","commits":[{"sha":"8c4190ca956a83f3286bc0d104041e94d53eaa59","author":{"email":"ae9e2289bfb86617e7df55ccf6837bd754e6357f@twopensource.com","name":"Nick Howard (Twitter)"},"message":"walk synthetic targets dependencies when constructing context.target()\n\nA recent change made it so that synthetic targets dependencies aren't traversed which caused their 3rdparty deps to be not added to the classpath in certain cases. This caused compilation errors. This patch retains the derivation checks from the previous one, but expands the synthetic targets after determining which ones are needed.\n\nRB of the patch that changed the ordering which caused the issue: https://rbcommons.com/s/twitter/r/1863/\n\nTesting Done:\nwrote unit test, verified existing behavior. ran context tests. CI baking at https://travis-ci.org/pantsbuild/pants/builds/54317247\n\nBugs closed: 1255\n\nReviewed at https://rbcommons.com/s/twitter/r/1914/","distinct":true,"url":"https://api.github.com/repos/benjyw/pants/commits/8c4190ca956a83f3286bc0d104041e94d53eaa59"}]}