KEMBAR78
Align with IDL dictionary changes by annevk · Pull Request #4745 · whatwg/html · GitHub
Skip to content

Conversation

@annevk
Copy link
Member

@annevk annevk commented Jul 1, 2019

@domenic
Copy link
Member

domenic commented Jul 2, 2019

This can be merged now, right? Or maybe we're concerned about breaking tools that automatically ingest the spec's IDL?

Is there a way that you found all the dictionaries that we could use to check your work, or would it be best to just assume you got them all?

@annevk
Copy link
Member Author

annevk commented Jul 3, 2019

I'm concerned about breakage. web-platform-tests consumes these IDL files.

I searched for "optional " and went through the 160 results.

Copy link
Member

@domenic domenic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I searched for [,(] ?optional.*\) and found one missing. Pushed, LGTM, but will let you figure out the best time to land based on tooling updates.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge yet Pull request must not be merged per rationale in comment

Development

Successfully merging this pull request may close these issues.

2 participants