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

Fix: #8. ld+yaml mediatype #9

Closed
wants to merge 1 commit into from
Closed

Fix: #8. ld+yaml mediatype #9

wants to merge 1 commit into from

Conversation

ioggstream
Copy link
Collaborator

This PR

  • registers the application/ld+yaml media type
  • registers the .yamlld file extension

@pchampin
Copy link

I like the idea of YAML-LD :)
Although it should is relatively straightforward, the devil is in the detail, and I believe some details should be explicitly ironed up. For example, IIRC, YAML is a superset of JSON. What is a processor supposed to do with YAML construct that do not map to JSON? Ignore them? Reject them as an error? Note that the answer may be different depending on where this happens (in data vs. in context).

@gkellogg
Copy link

As both @pchampin and @msporny mentioned (in #8 (comment)), it is important that the YAML used be restricted to just use native JSON features to insure backwards and forwards compatibility with JsON-LD.

@ioggstream ioggstream mentioned this pull request Jan 18, 2022
4 tasks
@darrelmiller darrelmiller linked an issue Mar 19, 2022 that may be closed by this pull request
4 tasks
@ioggstream
Copy link
Collaborator Author

Moved to another project: https://github.com/json-ld/yaml-ld

@ioggstream ioggstream closed this Jul 1, 2022
@VladimirAlexiev
Copy link

VladimirAlexiev commented Nov 10, 2024

Hi @ioggstream @pchampin @iherman!

Once resolved, we should also close w3c/json-ld-syntax#385

@ioggstream
Copy link
Collaborator Author

Hi @VladimirAlexiev. This PR has been closed because we decided to move the LD stuff to github.com/json-ld/yaml-ld :)

I will double-check the w3c/json-ld-syntax#385 to ensure that all registration parameters are ok.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ld+yaml media-type
4 participants