{"action":"created","issue":{"url":"https://api.github.com/repos/Origen-SDK/origen_testers/issues/52","repository_url":"https://api.github.com/repos/Origen-SDK/origen_testers","labels_url":"https://api.github.com/repos/Origen-SDK/origen_testers/issues/52/labels{/name}","comments_url":"https://api.github.com/repos/Origen-SDK/origen_testers/issues/52/comments","events_url":"https://api.github.com/repos/Origen-SDK/origen_testers/issues/52/events","html_url":"https://github.com/Origen-SDK/origen_testers/pull/52","id":256650694,"number":52,"title":"V93k flow optimizations","user":{"login":"ginty","id":158364,"avatar_url":"https://avatars3.githubusercontent.com/u/158364?v=4","gravatar_id":"","url":"https://api.github.com/users/ginty","html_url":"https://github.com/ginty","followers_url":"https://api.github.com/users/ginty/followers","following_url":"https://api.github.com/users/ginty/following{/other_user}","gists_url":"https://api.github.com/users/ginty/gists{/gist_id}","starred_url":"https://api.github.com/users/ginty/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ginty/subscriptions","organizations_url":"https://api.github.com/users/ginty/orgs","repos_url":"https://api.github.com/users/ginty/repos","events_url":"https://api.github.com/users/ginty/events{/privacy}","received_events_url":"https://api.github.com/users/ginty/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":18,"created_at":"2017-09-11T10:12:59Z","updated_at":"2017-09-25T16:42:10Z","closed_at":null,"author_association":"OWNER","pull_request":{"url":"https://api.github.com/repos/Origen-SDK/origen_testers/pulls/52","html_url":"https://github.com/Origen-SDK/origen_testers/pull/52","diff_url":"https://github.com/Origen-SDK/origen_testers/pull/52.diff","patch_url":"https://github.com/Origen-SDK/origen_testers/pull/52.patch"},"body":"This PR add the following V93K flow optimization improvements:\r\n\r\n* Bug fix whereby the test name uniqueness signature was not appended to the first test in the flow.\r\n* Improved the existing flag optimizer to make it recurse into nested groups and not only apply to the top-level\r\n* Added a new adjacent if optimizer to address #43, you can see it collapsing an existing adjacent if condition here: https://github.com/Origen-SDK/origen_testers/compare/v93k_flow_optimizations?expand=1#diff-2f5b6d997b1e017e3d495f5553aff516L2675\r\n* Added dedicated AST-level tests for the optimizers (plan to move these optimizers to the AST plugin at some point in the future)\r\n\r\nNote: before opening this PR, the prb1 flow in master was updated to move the main flow content to an imported component, this exposed the fact that the existing flag optimizations where not applied to nested groups.\r\nThe majority of the diffs in the approved files are restoring these flag optimizations.\r\n\r\n\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/Origen-SDK/origen_testers/issues/comments/331940646","html_url":"https://github.com/Origen-SDK/origen_testers/pull/52#issuecomment-331940646","issue_url":"https://api.github.com/repos/Origen-SDK/origen_testers/issues/52","id":331940646,"user":{"login":"coveralls","id":2354108,"avatar_url":"https://avatars1.githubusercontent.com/u/2354108?v=4","gravatar_id":"","url":"https://api.github.com/users/coveralls","html_url":"https://github.com/coveralls","followers_url":"https://api.github.com/users/coveralls/followers","following_url":"https://api.github.com/users/coveralls/following{/other_user}","gists_url":"https://api.github.com/users/coveralls/gists{/gist_id}","starred_url":"https://api.github.com/users/coveralls/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/coveralls/subscriptions","organizations_url":"https://api.github.com/users/coveralls/orgs","repos_url":"https://api.github.com/users/coveralls/repos","events_url":"https://api.github.com/users/coveralls/events{/privacy}","received_events_url":"https://api.github.com/users/coveralls/received_events","type":"User","site_admin":false},"created_at":"2017-09-25T16:42:10Z","updated_at":"2017-09-25T16:42:10Z","author_association":"NONE","body":"\n[![Coverage Status](https://coveralls.io/builds/13420193/badge)](https://coveralls.io/builds/13420193)\n\nCoverage increased (+0.3%) to 82.55% when pulling **80166e54f536275ce4e027436dc69256f2f3325b on v93k_flow_optimizations** into **bab3e4488d956bdcf43899cb9c37dc34fe29114e on master**.\n"}}
{"actor":{"display_login":"coveralls"}}