{"action":"created","issue":{"url":"https://api.github.com/repos/w3c/pub-manifest/issues/103","repository_url":"https://api.github.com/repos/w3c/pub-manifest","labels_url":"https://api.github.com/repos/w3c/pub-manifest/issues/103/labels{/name}","comments_url":"https://api.github.com/repos/w3c/pub-manifest/issues/103/comments","events_url":"https://api.github.com/repos/w3c/pub-manifest/issues/103/events","html_url":"https://github.com/w3c/pub-manifest/pull/103","id":504326948,"node_id":"MDExOlB1bGxSZXF1ZXN0MzI2MDAyNDQw","number":103,"title":"update manifest processing to use infra types","user":{"login":"mattgarrish","id":1565164,"node_id":"MDQ6VXNlcjE1NjUxNjQ=","avatar_url":"https://avatars0.githubusercontent.com/u/1565164?v=4","gravatar_id":"","url":"https://api.github.com/users/mattgarrish","html_url":"https://github.com/mattgarrish","followers_url":"https://api.github.com/users/mattgarrish/followers","following_url":"https://api.github.com/users/mattgarrish/following{/other_user}","gists_url":"https://api.github.com/users/mattgarrish/gists{/gist_id}","starred_url":"https://api.github.com/users/mattgarrish/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mattgarrish/subscriptions","organizations_url":"https://api.github.com/users/mattgarrish/orgs","repos_url":"https://api.github.com/users/mattgarrish/repos","events_url":"https://api.github.com/users/mattgarrish/events{/privacy}","received_events_url":"https://api.github.com/users/mattgarrish/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2019-10-08T23:18:57Z","updated_at":"2019-10-09T11:58:52Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/w3c/pub-manifest/pulls/103","html_url":"https://github.com/w3c/pub-manifest/pull/103","diff_url":"https://github.com/w3c/pub-manifest/pull/103.diff","patch_url":"https://github.com/w3c/pub-manifest/pull/103.patch"},"body":"Opening this as a PR to get feedback, as I'm going cross-eyed working on this section. Beyond just switching terminology, I also tried to tighten up the processing to account for potentially other bits of invalid input, to simplify the duplication of language/direction steps, and to consolidate a number of validation checks that were all dependent on recursion.\n\nThe WebIDL section is moved into the processing section, and to de-emphasize the technology in the interim I changed the title to \"Data Visualization\" (but other ideas welcome).\n\n\n<!--\n    This comment and the below content is programatically generated.\n    You may add a comma-separated list of anchors you'd like a\n    direct link to below (e.g. #idl-serializers, #idl-sequence):\n\n    Don't remove this comment or modify anything below this line.\n    If you don't want a preview generated for this pull request,\n    just replace the whole of this comment's content by \"no preview\"\n    and remove what's below.\n-->\n***\n<a href=\"https://pr-preview.s3.amazonaws.com/w3c/pub-manifest/pull/103.html\" title=\"Last updated on Oct 8, 2019, 11:19 PM UTC (e1532a4)\">Preview</a> | <a href=\"https://pr-preview.s3.amazonaws.com/w3c/pub-manifest/103/cd6bbd6...e1532a4.html\" title=\"Last updated on Oct 8, 2019, 11:19 PM UTC (e1532a4)\">Diff</a>"},"comment":{"url":"https://api.github.com/repos/w3c/pub-manifest/issues/comments/539969061","html_url":"https://github.com/w3c/pub-manifest/pull/103#issuecomment-539969061","issue_url":"https://api.github.com/repos/w3c/pub-manifest/issues/103","id":539969061,"node_id":"MDEyOklzc3VlQ29tbWVudDUzOTk2OTA2MQ==","user":{"login":"mattgarrish","id":1565164,"node_id":"MDQ6VXNlcjE1NjUxNjQ=","avatar_url":"https://avatars0.githubusercontent.com/u/1565164?v=4","gravatar_id":"","url":"https://api.github.com/users/mattgarrish","html_url":"https://github.com/mattgarrish","followers_url":"https://api.github.com/users/mattgarrish/followers","following_url":"https://api.github.com/users/mattgarrish/following{/other_user}","gists_url":"https://api.github.com/users/mattgarrish/gists{/gist_id}","starred_url":"https://api.github.com/users/mattgarrish/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mattgarrish/subscriptions","organizations_url":"https://api.github.com/users/mattgarrish/orgs","repos_url":"https://api.github.com/users/mattgarrish/repos","events_url":"https://api.github.com/users/mattgarrish/events{/privacy}","received_events_url":"https://api.github.com/users/mattgarrish/received_events","type":"User","site_admin":false},"created_at":"2019-10-09T11:58:52Z","updated_at":"2019-10-09T11:58:52Z","author_association":"MEMBER","body":"> For my taste the move towards infra did not make the spec more easy to read\r\n\r\nNo, that's what I've been struggling with the last couple of days... ;)\r\n\r\nBut by the end I did find it to be more precise than the rather loose references to javascript and generic object/arrays that we were using. Plus there is the benefit that if devs are already reading this notation in a [growing list of other specs](https://github.com/whatwg/infra/wiki/Dependent-specs), we're not expecting them to figure out something new.\r\n\r\n(I couldn't make sense of the final data without a cleaner representation, either.)"}}
{"actor":{"display_login":"mattgarrish"}}