{"action":"created","issue":{"url":"https://api.github.com/repos/openEHR/adl-antlr/issues/17","labels_url":"https://api.github.com/repos/openEHR/adl-antlr/issues/17/labels{/name}","comments_url":"https://api.github.com/repos/openEHR/adl-antlr/issues/17/comments","events_url":"https://api.github.com/repos/openEHR/adl-antlr/issues/17/events","html_url":"https://github.com/openEHR/adl-antlr/issues/17","id":113677096,"number":17,"title":"Things to do for Bert Verhees","user":{"login":"pieterbos","id":21126,"avatar_url":"https://avatars.githubusercontent.com/u/21126?v=3","gravatar_id":"","url":"https://api.github.com/users/pieterbos","html_url":"https://github.com/pieterbos","followers_url":"https://api.github.com/users/pieterbos/followers","following_url":"https://api.github.com/users/pieterbos/following{/other_user}","gists_url":"https://api.github.com/users/pieterbos/gists{/gist_id}","starred_url":"https://api.github.com/users/pieterbos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pieterbos/subscriptions","organizations_url":"https://api.github.com/users/pieterbos/orgs","repos_url":"https://api.github.com/users/pieterbos/repos","events_url":"https://api.github.com/users/pieterbos/events{/privacy}","received_events_url":"https://api.github.com/users/pieterbos/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-10-27T20:14:04Z","updated_at":"2015-10-28T09:31:16Z","closed_at":null,"body":"Since @BertVerhees asked for things to do, a list of suggestions of things he can do:\r\n\r\n- In the adl-anltr grammar (this project):\r\n   1. Fix the regular expression recognition in the ANTLR Lexer. The difficulty is that paths resemble regular expressions.\r\n\r\nIn archie (https://github.com/nedap/archie) :\r\n\r\n1. Implement tests for any language construct - you'll test this grammar as well as the parser. the framework is already in place\r\n2. Implement parsing of CTemporal types. The grammar should parse those already, but the treewalker does not."},"comment":{"url":"https://api.github.com/repos/openEHR/adl-antlr/issues/comments/151779376","html_url":"https://github.com/openEHR/adl-antlr/issues/17#issuecomment-151779376","issue_url":"https://api.github.com/repos/openEHR/adl-antlr/issues/17","id":151779376,"user":{"login":"pieterbos","id":21126,"avatar_url":"https://avatars.githubusercontent.com/u/21126?v=3","gravatar_id":"","url":"https://api.github.com/users/pieterbos","html_url":"https://github.com/pieterbos","followers_url":"https://api.github.com/users/pieterbos/followers","following_url":"https://api.github.com/users/pieterbos/following{/other_user}","gists_url":"https://api.github.com/users/pieterbos/gists{/gist_id}","starred_url":"https://api.github.com/users/pieterbos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pieterbos/subscriptions","organizations_url":"https://api.github.com/users/pieterbos/orgs","repos_url":"https://api.github.com/users/pieterbos/repos","events_url":"https://api.github.com/users/pieterbos/events{/privacy}","received_events_url":"https://api.github.com/users/pieterbos/received_events","type":"User","site_admin":false},"created_at":"2015-10-28T09:31:16Z","updated_at":"2015-10-28T09:31:16Z","body":"Hello Bert,\r\n\r\nthere are not really emails (although you can receive them this way), but github issues. You can find them on https://github.com/openEHR/adl-antlr/issues . \r\n\r\nhttps://github.com/openEHR/adl-antlr/issues/7 explains the regular expression problem.\r\n\r\nThe CTemporal types is not strange at all - i just have not implemented them yet, the grammar works fine and it is not an antlr problem - just missing java code.\r\n\r\nI think any testing in this case is useful. In any case, it's open source software, you can add anything you want in your own forks :) This grammar is mainly work by Thomas though, not me, I implemented a java library on top of it and tested the grammar that way.\r\n\r\n\r\n\r\n"}}