Add support for remote URIs in links #276
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
While working on the implementation for #273 it was unclear to me
whether the specification permits diagrams to be referenced directly by
URI or not so I split this out into its own pull request.
The specification specifically states:
This, to me, implied that we should be permitting absolute and relative
links via a URI to internet resources.
To implement this, a "wrapper"-ish function has been created for what
used to be a pure Back Matter lookup function. It handles the logic to
determine whether the link is internal (needs to be looked up in Back
Matter) or an internet resource (is a URI).
There are some incidental changes to code and tests due to the changing
of imports and addition of exported
test-data
.