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

[css-paint-api] Lean on "syntax definition" concept. #885

Merged
merged 2 commits into from Jul 23, 2019

Conversation

andruud
Copy link
Member

@andruud andruud commented May 6, 2019

The css-properties-values-api now describes in detail how to produce a
"syntax descriptor" from a string, so it's better to link to that rather
than linking to a section in the spec.

@andruud
Copy link
Member Author

andruud commented May 6, 2019

@bfgeek ptal

Is it possible to get newly "dfn export"-ed things to link locally? I can't get it to work ...

@tabatkins
Copy link
Member

Is it possible to get newly "dfn export"-ed things to link locally? I can't get it to work ...

You need to wait for Shepherd to make a pass over the defining spec (runs on each push, but can be delayed by other jobs, so 30-60 minutes is reasonable), then make sure you run bikeshed update locally to get the new definitions.

@andruud
Copy link
Member Author

andruud commented May 7, 2019

@tabatkins OK, in other words, this needs to be merged to master before I can bikeshed update, right?

@tabatkins
Copy link
Member

The properties-and-values definition needs to be in master before paint-api can use it, yeah.

(You can just commit the link now and it'll automatically reparse into a valid link later when things sync up.)

The css-properties-values-api now describes in detail how to produce a
"syntax descriptor" from a string, so it's better to link to that rather
than linking to a section in the spec.
@andruud andruud changed the title [css-paint-api] Lean on "syntax descriptor" concept. [css-paint-api] Lean on "syntax definition" concept. Jul 23, 2019
@tabatkins tabatkins merged commit 01ddded into w3c:master Jul 23, 2019
@andruud andruud deleted the paint/syntax_descriptor branch July 24, 2019 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants