{"action":"created","issue":{"url":"https://api.github.com/repos/theodi/csvlint/issues/186","labels_url":"https://api.github.com/repos/theodi/csvlint/issues/186/labels{/name}","comments_url":"https://api.github.com/repos/theodi/csvlint/issues/186/comments","events_url":"https://api.github.com/repos/theodi/csvlint/issues/186/events","html_url":"https://github.com/theodi/csvlint/issues/186","id":84202538,"number":186,"title":"Report an error if schema isn't loaded","user":{"login":"ldodds","id":109082,"avatar_url":"https://avatars.githubusercontent.com/u/109082?v=3","gravatar_id":"","url":"https://api.github.com/users/ldodds","html_url":"https://github.com/ldodds","followers_url":"https://api.github.com/users/ldodds/followers","following_url":"https://api.github.com/users/ldodds/following{/other_user}","gists_url":"https://api.github.com/users/ldodds/gists{/gist_id}","starred_url":"https://api.github.com/users/ldodds/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ldodds/subscriptions","organizations_url":"https://api.github.com/users/ldodds/orgs","repos_url":"https://api.github.com/users/ldodds/repos","events_url":"https://api.github.com/users/ldodds/events{/privacy}","received_events_url":"https://api.github.com/users/ldodds/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/theodi/csvlint/labels/bug","name":"bug","color":"fc2929"},{"url":"https://api.github.com/repos/theodi/csvlint/labels/ready","name":"ready","color":"ededed"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2015-06-02T20:41:27Z","updated_at":"2015-07-10T13:38:33Z","closed_at":null,"body":"If a user supplies a schema and we fail to load it, then the application should report an error so its clear that the schema hasn't been applied.\r\n\r\nCurrently it falls back to not applying any schema validation which is misleading.\r\n\r\nAs a first step, reporting a basic error would be an immediate improvement. \r\n\r\nReporting why the schema is invalid would also be useful, although there are a range of possible errors:\r\n\r\n* invalid JSON\r\n* invalid regex\r\n* invalid values for a constraint specification, e.g. a number or string where true/false is expected\r\n* unknown constraint\r\n\r\nThe first two are definitely fatal errors. The last two are potentially just warnings. Depends if we want to work on a \"best effort\" basis.\r\n"},"comment":{"url":"https://api.github.com/repos/theodi/csvlint/issues/comments/120413934","html_url":"https://github.com/theodi/csvlint/issues/186#issuecomment-120413934","issue_url":"https://api.github.com/repos/theodi/csvlint/issues/186","id":120413934,"user":{"login":"quadrophobiac","id":1039017,"avatar_url":"https://avatars.githubusercontent.com/u/1039017?v=3","gravatar_id":"","url":"https://api.github.com/users/quadrophobiac","html_url":"https://github.com/quadrophobiac","followers_url":"https://api.github.com/users/quadrophobiac/followers","following_url":"https://api.github.com/users/quadrophobiac/following{/other_user}","gists_url":"https://api.github.com/users/quadrophobiac/gists{/gist_id}","starred_url":"https://api.github.com/users/quadrophobiac/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/quadrophobiac/subscriptions","organizations_url":"https://api.github.com/users/quadrophobiac/orgs","repos_url":"https://api.github.com/users/quadrophobiac/repos","events_url":"https://api.github.com/users/quadrophobiac/events{/privacy}","received_events_url":"https://api.github.com/users/quadrophobiac/received_events","type":"User","site_admin":false},"created_at":"2015-07-10T13:38:33Z","updated_at":"2015-07-10T13:38:33Z","body":"I've confirmed non triggering by running the tests suite with a byebug in each conditional"}}