Skip to content

RFC for parse result link relations#23

Merged
zdne merged 1 commit intomasterfrom
smizell/parse-result-link-relations
Nov 26, 2015
Merged

RFC for parse result link relations#23
zdne merged 1 commit intomasterfrom
smizell/parse-result-link-relations

Conversation

@smizell
Copy link
Contributor

@smizell smizell commented Nov 24, 2015

This is an RFC for link relations for the parse result namespace.

@danielgtaylor
Copy link
Contributor

Distilling my comments from our Slack conversation:

  1. I am still a little uncomfortable with the inferred link relation. It just doesn't feel like a link to me, but I won't block this because I don't have a better solution that is as generic as this one. A property on meta called inferred is very specific to this particular use case.
  2. The origin relation makes sense to me since it is a link to the originating software. I like this.

So, given we get some other eyes on this (@Almad, @zdne, etc.) then I am okay with merging.

@zdne
Copy link
Member

zdne commented Nov 25, 2015

I am OK with this once #22 is resolved.

@danielgtaylor

It just doesn't feel like a link to me,

I think it is good if the inferred points to the author (tool) of the inferred element, so I think it could (read should) be a link

@zdne
Copy link
Member

zdne commented Nov 26, 2015

Merging with #22 being active

zdne added a commit that referenced this pull request Nov 26, 2015
…relations

RFC for parse result link relations
@zdne zdne merged commit 13b7466 into master Nov 26, 2015
@zdne zdne deleted the smizell/parse-result-link-relations branch November 26, 2015 10:25
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