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

Clarify that expansion steps looking for entries #184

Merged
merged 3 commits into from Oct 31, 2019
Merged

Clarify that expansion steps looking for entries #184

merged 3 commits into from Oct 31, 2019

Conversation

@gkellogg
Copy link
Member

gkellogg commented Oct 30, 2019

only apply when result is a map.

Also, split-up and remove final step of Expansion Algorithm.

Fixes #175 and fixes #183.

cc/ @kasei


Preview | Diff

…ult_ is a map.

Fixes #183.
@gkellogg gkellogg requested a review from pchampin Oct 30, 2019
@gkellogg gkellogg merged commit da9b2f0 into master Oct 31, 2019
2 of 3 checks passed
2 of 3 checks passed
Travis CI - Pull Request Build Errored
Details
Travis CI - Branch Build Passed
Details
ipr PR deemed acceptable.
Details
@gkellogg gkellogg deleted the issue-175 branch Oct 31, 2019
@kasei

This comment has been minimized.

Copy link

kasei commented Oct 31, 2019

Thanks @gkellogg . Just FYI, you're tagging someone else in that cc. s/ie/ei/

kasei added a commit to kasei/perl-jsonld that referenced this pull request Dec 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.