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

Fixed returned @context issues when the context is a remote context #296

Merged
merged 6 commits into from
Sep 23, 2020

Conversation

umbreak
Copy link
Contributor

@umbreak umbreak commented Sep 14, 2020

Fixes #295
Fixes #184
Fixes #297
Fixes #298

@umbreak umbreak force-pushed the compact_with_iri branch 2 times, most recently from 066c3db to 0148848 Compare September 14, 2020 18:03
@coveralls
Copy link

coveralls commented Sep 14, 2020

Coverage Status

Coverage increased (+0.2%) to 89.56% when pulling 4894387 on umbreak:compact_with_iri into 8001da1 on jsonld-java:master.

@umbreak umbreak changed the title Returned context IRI after compaction Returned context IRI after compaction and framing Sep 15, 2020
@umbreak umbreak changed the title Returned context IRI after compaction and framing Fixed returned @context issues Sep 15, 2020
@umbreak umbreak changed the title Fixed returned @context issues Fixed returned @context issues when the context is a remote context Sep 15, 2020
@umbreak
Copy link
Contributor Author

umbreak commented Sep 15, 2020

As far as I have seen in the spec and in the tests done in the playground, the passed context is the one that has to be provided as a result, not the resolved one or anything else.

It still passes the tests from the W3C suite, so I assume there is no further problem with it.

dr0i added a commit to dr0i/jsonld-java that referenced this pull request Sep 22, 2020
Copy link
Member

@dr0i dr0i left a comment

Choose a reason for hiding this comment

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

First of all: thx a lot for your contribution. This fixes a bunch of issues which is really great!

Note that I opened a PR against your branch, just some minor cleanup.

@umbreak umbreak requested a review from dr0i September 22, 2020 17:34
@umbreak
Copy link
Contributor Author

umbreak commented Sep 22, 2020

I've added the changes from your PR in my latest commit

Copy link
Member

@ansell ansell left a comment

Choose a reason for hiding this comment

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

Needs some clarification about the removal of the Context.serialize method.

@umbreak umbreak requested a review from ansell September 23, 2020 17:15
@ansell ansell merged commit ad940df into jsonld-java:master Sep 23, 2020
@ansell
Copy link
Member

ansell commented Sep 23, 2020

Looks good, thanks. Will release today.

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