{"action":"created","issue":{"url":"https://api.github.com/repos/spacetelescope/asdf-standard/issues/78","labels_url":"https://api.github.com/repos/spacetelescope/asdf-standard/issues/78/labels{/name}","comments_url":"https://api.github.com/repos/spacetelescope/asdf-standard/issues/78/comments","events_url":"https://api.github.com/repos/spacetelescope/asdf-standard/issues/78/events","html_url":"https://github.com/spacetelescope/asdf-standard/pull/78","id":66227122,"number":78,"title":"flow_style -> flowStyle for consistency with camelCase of other keywords","user":{"login":"embray","id":676149,"avatar_url":"https://avatars.githubusercontent.com/u/676149?v=3","gravatar_id":"","url":"https://api.github.com/users/embray","html_url":"https://github.com/embray","followers_url":"https://api.github.com/users/embray/followers","following_url":"https://api.github.com/users/embray/following{/other_user}","gists_url":"https://api.github.com/users/embray/gists{/gist_id}","starred_url":"https://api.github.com/users/embray/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/embray/subscriptions","organizations_url":"https://api.github.com/users/embray/orgs","repos_url":"https://api.github.com/users/embray/repos","events_url":"https://api.github.com/users/embray/events{/privacy}","received_events_url":"https://api.github.com/users/embray/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-04-03T21:02:19Z","updated_at":"2015-04-06T16:51:14Z","closed_at":"2015-04-06T16:19:07Z","pull_request":{"url":"https://api.github.com/repos/spacetelescope/asdf-standard/pulls/78","html_url":"https://github.com/spacetelescope/asdf-standard/pull/78","diff_url":"https://github.com/spacetelescope/asdf-standard/pull/78.diff","patch_url":"https://github.com/spacetelescope/asdf-standard/pull/78.patch"},"body":"If any associated changes need to be made in pyasdf I'll do that too.\r\n\r\n<s>Relatedly, I was reading the descriptions of this, and the `style` keyword, and thought of adding a note that implementations *may* check this on reading too.  It would be up to implementations decide how to alert about objects that are not given in the specified style, though the recommended default is to ignore it.</s>\r\n\r\n<s>I don't know, what do you think?  I feel like in general this should be ignored on reading, but I think there could also be value in using this for checking a file to *strict* adherence to convention, and that implementations should at least know that they have that option.  I believe this is possible to do within pyyaml but I'm not certain.</s>\r\n\r\nOn second thought, scratch all that.  That has the same problem as trying to preserve comments, I guess.  This would have to hook schema validation deep into the YAML parser in a way that might be doable but probably isn't practical in the current model."},"comment":{"url":"https://api.github.com/repos/spacetelescope/asdf-standard/issues/comments/90151334","html_url":"https://github.com/spacetelescope/asdf-standard/pull/78#issuecomment-90151334","issue_url":"https://api.github.com/repos/spacetelescope/asdf-standard/issues/78","id":90151334,"user":{"login":"mdboom","id":38294,"avatar_url":"https://avatars.githubusercontent.com/u/38294?v=3","gravatar_id":"","url":"https://api.github.com/users/mdboom","html_url":"https://github.com/mdboom","followers_url":"https://api.github.com/users/mdboom/followers","following_url":"https://api.github.com/users/mdboom/following{/other_user}","gists_url":"https://api.github.com/users/mdboom/gists{/gist_id}","starred_url":"https://api.github.com/users/mdboom/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mdboom/subscriptions","organizations_url":"https://api.github.com/users/mdboom/orgs","repos_url":"https://api.github.com/users/mdboom/repos","events_url":"https://api.github.com/users/mdboom/events{/privacy}","received_events_url":"https://api.github.com/users/mdboom/received_events","type":"User","site_admin":false},"created_at":"2015-04-06T16:51:14Z","updated_at":"2015-04-06T16:51:14Z","body":"There are examples that test the example schema against the metaschema, but they don't fail because extra keys are allowed in the JSON metaschema.  So `flowStyle` isn't rejected, even though it was defined as `flow_style`.  A test that checked that expected a failure when `flowStyle` is some incorrect value would have caught this, though."}}