Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Align with IDL dictionary changes #4745

Merged
merged 2 commits into from
Aug 9, 2019
Merged

Align with IDL dictionary changes #4745

merged 2 commits into from
Aug 9, 2019

Conversation

@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