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

Add @protected & @version #110

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

botris
Copy link

@botris botris commented Oct 2, 2023

Json LD defines both protected and version, this PR adds both.

@k00ni
Copy link
Contributor

k00ni commented Oct 3, 2023

Good addition. Please add fixes #111 or add #111 in Development tab (right side), so it gets closed when this PR is merged.

@k00ni
Copy link
Contributor

k00ni commented Oct 10, 2023

Ping @lanthaler

@lanthaler
Copy link
Owner

lanthaler commented Oct 12, 2023

JsonLD is a 1.0 compliant parser, not 1.1. These changes break 1.0 compliance but don't make it 1.1 compliant as many of the new features will remain unsupported.

We thus unfortunately can't merge this in the current form. We either need to hide this behind a configuration flag or extend this PR to make JsonLD fully compliant with JSON-LD 1.1.

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