{"action":"created","issue":{"url":"https://api.github.com/repos/openwdl/wdl/issues/231","repository_url":"https://api.github.com/repos/openwdl/wdl","labels_url":"https://api.github.com/repos/openwdl/wdl/issues/231/labels{/name}","comments_url":"https://api.github.com/repos/openwdl/wdl/issues/231/comments","events_url":"https://api.github.com/repos/openwdl/wdl/issues/231/events","html_url":"https://github.com/openwdl/wdl/pull/231","id":341047340,"node_id":"MDExOlB1bGxSZXF1ZXN0MjAxMzEzMTg1","number":231,"title":"Add task to the list of scopes","user":{"login":"pshapiro4broad","id":1799360,"node_id":"MDQ6VXNlcjE3OTkzNjA=","avatar_url":"https://avatars2.githubusercontent.com/u/1799360?v=4","gravatar_id":"","url":"https://api.github.com/users/pshapiro4broad","html_url":"https://github.com/pshapiro4broad","followers_url":"https://api.github.com/users/pshapiro4broad/followers","following_url":"https://api.github.com/users/pshapiro4broad/following{/other_user}","gists_url":"https://api.github.com/users/pshapiro4broad/gists{/gist_id}","starred_url":"https://api.github.com/users/pshapiro4broad/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pshapiro4broad/subscriptions","organizations_url":"https://api.github.com/users/pshapiro4broad/orgs","repos_url":"https://api.github.com/users/pshapiro4broad/repos","events_url":"https://api.github.com/users/pshapiro4broad/events{/privacy}","received_events_url":"https://api.github.com/users/pshapiro4broad/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":7,"created_at":"2018-07-13T14:47:13Z","updated_at":"2018-07-17T14:38:31Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/openwdl/wdl/pulls/231","html_url":"https://github.com/openwdl/wdl/pull/231","diff_url":"https://github.com/openwdl/wdl/pull/231.diff","patch_url":"https://github.com/openwdl/wdl/pull/231.patch"},"body":"If section \"Scope\" lists all WDL scopes, it must include `task`.\r\n\r\nAlso, I think the code sample was in error, in that it was providing `var` to `my_task` but `var` wasn't a declared input."},"comment":{"url":"https://api.github.com/repos/openwdl/wdl/issues/comments/405605547","html_url":"https://github.com/openwdl/wdl/pull/231#issuecomment-405605547","issue_url":"https://api.github.com/repos/openwdl/wdl/issues/231","id":405605547,"node_id":"MDEyOklzc3VlQ29tbWVudDQwNTYwNTU0Nw==","user":{"login":"geoffjentry","id":961771,"node_id":"MDQ6VXNlcjk2MTc3MQ==","avatar_url":"https://avatars3.githubusercontent.com/u/961771?v=4","gravatar_id":"","url":"https://api.github.com/users/geoffjentry","html_url":"https://github.com/geoffjentry","followers_url":"https://api.github.com/users/geoffjentry/followers","following_url":"https://api.github.com/users/geoffjentry/following{/other_user}","gists_url":"https://api.github.com/users/geoffjentry/gists{/gist_id}","starred_url":"https://api.github.com/users/geoffjentry/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/geoffjentry/subscriptions","organizations_url":"https://api.github.com/users/geoffjentry/orgs","repos_url":"https://api.github.com/users/geoffjentry/repos","events_url":"https://api.github.com/users/geoffjentry/events{/privacy}","received_events_url":"https://api.github.com/users/geoffjentry/received_events","type":"User","site_admin":false},"created_at":"2018-07-17T14:38:31Z","updated_at":"2018-07-17T14:38:31Z","author_association":"MEMBER","body":"@EvanTheB re the grammar file, it's murky as there's no official statement as to which is the ground truth. IMO in a perfect world there's a grammar file which is the official source of truth and then a prose spec to flesh it out. However I don't think the hermes grammar is appropriate for that as very few people understand hermes well enough to make any meaningful changes. \r\n\r\nI've been hoping some motivated individual would submit a BNF or something like that, but that hasn't yet happened. I personally view the hermes grammar as a sorta/kinda reference implementation of the spec - I'm only aware of one WDL implementation not using the hermes grammar (the IntelliJ plugin) so for all intents and purposes the hermes grammar **is** the reality that people experience. \r\n\r\nre the grammar snippets in the spec doc, I've **never** liked having those there as that too tightly binds the prose w/ the specific grammar used. IMO whenever folks are putting together PRs in areas where the grammar is directly referenced in the spec file it should be removed in favor of real examples.\r\n\r\n"}}
{"actor":{"display_login":"geoffjentry"}}