{"action":"created","issue":{"url":"https://api.github.com/repos/wyvernlang/wyvern/issues/126","repository_url":"https://api.github.com/repos/wyvernlang/wyvern","labels_url":"https://api.github.com/repos/wyvernlang/wyvern/issues/126/labels{/name}","comments_url":"https://api.github.com/repos/wyvernlang/wyvern/issues/126/comments","events_url":"https://api.github.com/repos/wyvernlang/wyvern/issues/126/events","html_url":"https://github.com/wyvernlang/wyvern/pull/126","id":176158773,"number":126,"title":"Add transformation over IL programs which makes casts to/from Dyn typesafe","user":{"login":"craigthelinguist","id":5293283,"avatar_url":"https://avatars.githubusercontent.com/u/5293283?v=3","gravatar_id":"","url":"https://api.github.com/users/craigthelinguist","html_url":"https://github.com/craigthelinguist","followers_url":"https://api.github.com/users/craigthelinguist/followers","following_url":"https://api.github.com/users/craigthelinguist/following{/other_user}","gists_url":"https://api.github.com/users/craigthelinguist/gists{/gist_id}","starred_url":"https://api.github.com/users/craigthelinguist/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/craigthelinguist/subscriptions","organizations_url":"https://api.github.com/users/craigthelinguist/orgs","repos_url":"https://api.github.com/users/craigthelinguist/repos","events_url":"https://api.github.com/users/craigthelinguist/events{/privacy}","received_events_url":"https://api.github.com/users/craigthelinguist/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2016-09-10T05:18:48Z","updated_at":"2016-09-26T06:41:47Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/wyvernlang/wyvern/pulls/126","html_url":"https://github.com/wyvernlang/wyvern/pull/126","diff_url":"https://github.com/wyvernlang/wyvern/pull/126.diff","patch_url":"https://github.com/wyvernlang/wyvern/pull/126.patch"},"body":"Addresses #87 \r\n\r\nKey changes:\r\n* Code in MethodCall has been cleaned up.\r\n* Add a new ILTransform interface which operates on IExpr.\r\n* Implemented some of the functionality for a transformation which makes the use of Dyn type-safe at runtime.\r\n* Add new testing framework with two basic tests to give an example of how the transformation works.\r\n\r\nMore to come."},"comment":{"url":"https://api.github.com/repos/wyvernlang/wyvern/issues/comments/249490927","html_url":"https://github.com/wyvernlang/wyvern/pull/126#issuecomment-249490927","issue_url":"https://api.github.com/repos/wyvernlang/wyvern/issues/126","id":249490927,"user":{"login":"craigthelinguist","id":5293283,"avatar_url":"https://avatars.githubusercontent.com/u/5293283?v=3","gravatar_id":"","url":"https://api.github.com/users/craigthelinguist","html_url":"https://github.com/craigthelinguist","followers_url":"https://api.github.com/users/craigthelinguist/followers","following_url":"https://api.github.com/users/craigthelinguist/following{/other_user}","gists_url":"https://api.github.com/users/craigthelinguist/gists{/gist_id}","starred_url":"https://api.github.com/users/craigthelinguist/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/craigthelinguist/subscriptions","organizations_url":"https://api.github.com/users/craigthelinguist/orgs","repos_url":"https://api.github.com/users/craigthelinguist/repos","events_url":"https://api.github.com/users/craigthelinguist/events{/privacy}","received_events_url":"https://api.github.com/users/craigthelinguist/received_events","type":"User","site_admin":false},"created_at":"2016-09-26T06:41:47Z","updated_at":"2016-09-26T06:41:47Z","body":"Thanks for the feedback! I've attempted those changes and managed to get the two (very basic) tests passing.\r\n\r\nI dropped the ILTransformer interface, because the abstraction it captures is the same as ASTVisitor<GenContext, ASTNode> so I don't personally see the worth of having that be separate. If you have different thoughts on the matter, I can add that back in though."}}
{"actor":{"display_login":"craigthelinguist"}}