{"action":"created","issue":{"url":"https://api.github.com/repos/schyntax/schyntax/issues/7","labels_url":"https://api.github.com/repos/schyntax/schyntax/issues/7/labels{/name}","comments_url":"https://api.github.com/repos/schyntax/schyntax/issues/7/comments","events_url":"https://api.github.com/repos/schyntax/schyntax/issues/7/events","html_url":"https://github.com/schyntax/schyntax/pull/7","id":107067779,"number":7,"title":"add parse and validation errors to JSON suite","user":{"login":"nchidsey","id":14133223,"avatar_url":"https://avatars.githubusercontent.com/u/14133223?v=3","gravatar_id":"","url":"https://api.github.com/users/nchidsey","html_url":"https://github.com/nchidsey","followers_url":"https://api.github.com/users/nchidsey/followers","following_url":"https://api.github.com/users/nchidsey/following{/other_user}","gists_url":"https://api.github.com/users/nchidsey/gists{/gist_id}","starred_url":"https://api.github.com/users/nchidsey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nchidsey/subscriptions","organizations_url":"https://api.github.com/users/nchidsey/orgs","repos_url":"https://api.github.com/users/nchidsey/repos","events_url":"https://api.github.com/users/nchidsey/events{/privacy}","received_events_url":"https://api.github.com/users/nchidsey/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-09-17T20:28:39Z","updated_at":"2015-09-25T04:13:26Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/schyntax/schyntax/pulls/7","html_url":"https://github.com/schyntax/schyntax/pull/7","diff_url":"https://github.com/schyntax/schyntax/pull/7.diff","patch_url":"https://github.com/schyntax/schyntax/pull/7.patch"},"body":"Relating to [Issue 6](https://github.com/schyntax/schyntax/issues/6), this adds JSON structure support and test cases for some parse and validation errors.  So far, this branch does not include changes for testing `ValidTimeNotFoundError`s.  As mentioned in the issue, this currently has a parse error index of 0 for empty schedules, though the index could change.  Also worth mentioning that the reference implementation might conflict on at least one, `minute(monday)`."},"comment":{"url":"https://api.github.com/repos/schyntax/schyntax/issues/comments/143119832","html_url":"https://github.com/schyntax/schyntax/pull/7#issuecomment-143119832","issue_url":"https://api.github.com/repos/schyntax/schyntax/issues/7","id":143119832,"user":{"login":"bretcope","id":3608260,"avatar_url":"https://avatars.githubusercontent.com/u/3608260?v=3","gravatar_id":"","url":"https://api.github.com/users/bretcope","html_url":"https://github.com/bretcope","followers_url":"https://api.github.com/users/bretcope/followers","following_url":"https://api.github.com/users/bretcope/following{/other_user}","gists_url":"https://api.github.com/users/bretcope/gists{/gist_id}","starred_url":"https://api.github.com/users/bretcope/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bretcope/subscriptions","organizations_url":"https://api.github.com/users/bretcope/orgs","repos_url":"https://api.github.com/users/bretcope/repos","events_url":"https://api.github.com/users/bretcope/events{/privacy}","received_events_url":"https://api.github.com/users/bretcope/received_events","type":"User","site_admin":false},"created_at":"2015-09-25T04:13:26Z","updated_at":"2015-09-25T04:13:26Z","body":"This looks good, with one exception. The `parseErrorIndex` for `dates(1/-1)` should be 8 instead of 6 because the error is on the minus sign, not the start of the date value. I'll merge this and make that fix.\r\n\r\nI have the C# implementation ready to go. I'll get the others setup soon."}}