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

Update menclose spec URL #11163

Merged
merged 2 commits into from
Jun 20, 2021
Merged

Conversation

sideshowbarker
Copy link
Contributor

@sideshowbarker sideshowbarker requested a review from Elchi3 June 20, 2021 01:04
@github-actions github-actions bot added the data:mathml Compat data for MathML features. https://developer.mozilla.org/docs/Web/MathML label Jun 20, 2021
@foolip
Copy link
Contributor

foolip commented Jun 20, 2021

CI is failing because this URL isn't in browser-specs.

@sideshowbarker sideshowbarker requested a review from ddbeck as a code owner June 20, 2021 09:39
@github-actions github-actions bot added the linter Issues or pull requests regarding the tests / linter of the JSON files. label Jun 20, 2021
@sideshowbarker
Copy link
Contributor Author

CI is failing because this URL isn't in browser-specs.

oops — fixed now

@foolip foolip merged commit 56c6559 into mdn:main Jun 20, 2021
@foolip
Copy link
Contributor

foolip commented Jun 20, 2021

Oops, this isn't strictly data and I shouldn't have merged. @Elchi3 or @ddbeck for post-merge review.

@Elchi3
Copy link
Member

Elchi3 commented Jun 21, 2021

r+

browser-specs has
https://w3c.github.io/mathml-aam
https://w3c.github.io/mathml-core
but not
https://w3c.github.io/mathml

menclose might make it into MathML-core, see w3c/mathml-core#274, so exception list sounds fine to me for the moment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:mathml Compat data for MathML features. https://developer.mozilla.org/docs/Web/MathML linter Issues or pull requests regarding the tests / linter of the JSON files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants