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

(closes #119) Introduce extractAllScripts flag to handle YAML streams #133

Merged
merged 5 commits into from
Apr 4, 2024

Conversation

anatoly-scherbakov
Copy link
Contributor

@anatoly-scherbakov anatoly-scherbakov commented Feb 18, 2024

Why

We do not support YAML Streams, but we already have some support for them within <script> tags.

What

Generalize that support to YAML-LD files too.


Preview | Diff

@anatoly-scherbakov anatoly-scherbakov changed the base branch from main to 131-yaml-under-script-html-tag February 18, 2024 21:02
Copy link
Contributor

@TallTed TallTed left a comment

Choose a reason for hiding this comment

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

small things

Base automatically changed from 131-yaml-under-script-html-tag to main April 3, 2024 20:02
@gkellogg
Copy link
Member

gkellogg commented Apr 3, 2024

The JSON-LD WG discussed this and decided that work can proceed on the editor's draft, so this can be merged after reconciled.

@anatoly-scherbakov anatoly-scherbakov force-pushed the 119-extract-all-streams branch from 4119bcc to 9f9b6ef Compare April 4, 2024 17:57
@anatoly-scherbakov anatoly-scherbakov merged commit 06f0245 into main Apr 4, 2024
2 checks passed
@gkellogg gkellogg deleted the 119-extract-all-streams branch June 12, 2024 20:33
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.

3 participants