{"action":"created","issue":{"url":"https://api.github.com/repos/Bradfield/languages-2017-01/issues/1","repository_url":"https://api.github.com/repos/Bradfield/languages-2017-01","labels_url":"https://api.github.com/repos/Bradfield/languages-2017-01/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/Bradfield/languages-2017-01/issues/1/comments","events_url":"https://api.github.com/repos/Bradfield/languages-2017-01/issues/1/events","html_url":"https://github.com/Bradfield/languages-2017-01/pull/1","id":201529276,"number":1,"title":"[Work in Progress] Add complex expressions","user":{"login":"richardartoul","id":9171254,"avatar_url":"https://avatars.githubusercontent.com/u/9171254?v=3","gravatar_id":"","url":"https://api.github.com/users/richardartoul","html_url":"https://github.com/richardartoul","followers_url":"https://api.github.com/users/richardartoul/followers","following_url":"https://api.github.com/users/richardartoul/following{/other_user}","gists_url":"https://api.github.com/users/richardartoul/gists{/gist_id}","starred_url":"https://api.github.com/users/richardartoul/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/richardartoul/subscriptions","organizations_url":"https://api.github.com/users/richardartoul/orgs","repos_url":"https://api.github.com/users/richardartoul/repos","events_url":"https://api.github.com/users/richardartoul/events{/privacy}","received_events_url":"https://api.github.com/users/richardartoul/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2017-01-18T10:10:30Z","updated_at":"2017-01-18T21:44:13Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Bradfield/languages-2017-01/pulls/1","html_url":"https://github.com/Bradfield/languages-2017-01/pull/1","diff_url":"https://github.com/Bradfield/languages-2017-01/pull/1.diff","patch_url":"https://github.com/Bradfield/languages-2017-01/pull/1.patch"},"body":"node test_runner.js to run the test suite.\r\n\r\nAll the tests pass but the parser is complaining about ambiguity. Any ideas?\r\n\r\n```\r\nConflict in grammar: multiple actions possible when lookahead token is CPAREN in state 34\r\n- reduce by rule: arguments -> exp\r\n- shift token (then go to state 38)\r\nConflict in grammar: multiple actions possible when lookahead token is MINUS in state 37\r\n- reduce by rule: exp -> exp operator exp\r\n- shift token (then go to state 30)\r\nConflict in grammar: multiple actions possible when lookahead token is PLUS in state 37\r\n- reduce by rule: exp -> exp operator exp\r\n- shift token (then go to state 31)\r\nConflict in grammar: multiple actions possible when lookahead token is MULTIPLY in state 37\r\n- reduce by rule: exp -> exp operator exp\r\n- shift token (then go to state 32)\r\nConflict in grammar: multiple actions possible when lookahead token is DIVIDE in state 37\r\n- reduce by rule: exp -> exp operator exp\r\n- shift token (then go to state 33)\r\n\r\nStates with conflicts:\r\nState 34\r\n  exp -> OPAREN exp .CPAREN #lookaheads= CPAREN MINUS PLUS MULTIPLY DIVIDE COMMA SEMICOLON\r\n  exp -> exp .operator exp\r\n  arguments -> exp . #lookaheads= CPAREN COMMA\r\n  operator -> .MINUS\r\n  operator -> .PLUS\r\n  operator -> .MULTIPLY\r\n  operator -> .DIVIDE\r\nState 37\r\n  exp -> exp operator exp . #lookaheads= CPAREN MINUS PLUS MULTIPLY DIVIDE COMMA SEMICOLON\r\n  exp -> exp .operator exp\r\n  operator -> .MINUS\r\n  operator -> .PLUS\r\n  operator -> .MULTIPLY\r\n  operator -> .DIVIDE\r\n```"},"comment":{"url":"https://api.github.com/repos/Bradfield/languages-2017-01/issues/comments/273611209","html_url":"https://github.com/Bradfield/languages-2017-01/pull/1#issuecomment-273611209","issue_url":"https://api.github.com/repos/Bradfield/languages-2017-01/issues/1","id":273611209,"user":{"login":"quackingduck","id":9556,"avatar_url":"https://avatars.githubusercontent.com/u/9556?v=3","gravatar_id":"","url":"https://api.github.com/users/quackingduck","html_url":"https://github.com/quackingduck","followers_url":"https://api.github.com/users/quackingduck/followers","following_url":"https://api.github.com/users/quackingduck/following{/other_user}","gists_url":"https://api.github.com/users/quackingduck/gists{/gist_id}","starred_url":"https://api.github.com/users/quackingduck/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/quackingduck/subscriptions","organizations_url":"https://api.github.com/users/quackingduck/orgs","repos_url":"https://api.github.com/users/quackingduck/repos","events_url":"https://api.github.com/users/quackingduck/events{/privacy}","received_events_url":"https://api.github.com/users/quackingduck/received_events","type":"User","site_admin":false},"created_at":"2017-01-18T21:44:13Z","updated_at":"2017-01-18T21:44:13Z","body":"(Can you make future commits in a branch on this repo so we don't have to add another remote?)\r\n\r\nIt's not clear to me what's going on with the conflict in the grammar. But from an AST design perspective. We want an `expression` rule but not necessarily an expression _node_. An expression is broadly \"anything that evaluates to a value\" while a statement is \"something run for its side-effects, and that doesn't evaluate to anything\""}}
{"actor":{"display_login":"quackingduck"}}