Skip to content
This repository has been archived by the owner on Jul 30, 2019. It is now read-only.

Fix #987 #1186

Merged
merged 14 commits into from
Feb 3, 2018
Merged

Fix #987 #1186

merged 14 commits into from
Feb 3, 2018

Conversation

brucelawson
Copy link

No description provided.

Copy link
Collaborator

@chaals chaals left a comment

Choose a reason for hiding this comment

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

*These need to be added to the changelog. (That's the change requested before merge).
*Somehow they got piled onto an old branch, instead of through a new branch based on master. That seems to happen sometimes, but it makes life complex. On the other hand it shouldn't affect this particular issue so let's deal with it offline

  • Think it would help to be clearer whether resource-hints or this spec is considered normative from the perspective of HTML. Maybe have non-normative content describing the things that are normatively defined elsewhere - but it needs a normative reference...

But mostly looks good - thanks!

@chaals chaals merged commit e684ce4 into master Feb 3, 2018
@brucelawson brucelawson deleted the bruce-add-link-types-987 branch February 3, 2018 16:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants