Skip to content

Conversation

@rubys
Copy link
Contributor

@rubys rubys commented Aug 8, 2018

No description provided.

wooorm added a commit to syntax-tree/mdast that referenced this pull request Aug 31, 2018
This change adds support for a new optional field on the association
mixin, used by `definition`, `footnoteDefinition`, the `Reference`
mixin (in turn used by `linkReference`, `imageReference`, and
`footnoteReference`).

The value of the `label` field must, when present, hold the
non-normalised value of the `identifier` field.

This allows references that are not association with definitions
to be represented correctly as raw text again.

Closes GH-23.
Related to syntax-tree/mdast-util-to-hast#20
Related to syntax-tree/mdast-util-to-hast#21
Related to syntax-tree/mdast-util-to-hast#22
Related to remarkjs/remark#346
@BridgeAR
Copy link

BridgeAR commented Sep 5, 2018

Ping @wooorm

@rubys
Copy link
Contributor Author

rubys commented Sep 5, 2018

@BridgeAR: @woorm is waiting on me. I'll try to clean this up in the next 24 hours. Sorry for the delay.

wooorm added a commit to remarkjs/remark that referenced this pull request Oct 14, 2018
* remark-parse: add label to definition, footnote definition, link reference, image reference,
  footnote reference
* remark-stringify: use full reference if no type is set

Related to syntax-tree/mdast#23.
Related to syntax-tree/mdast-util-to-hast#22.

Supersedes GH-346.

Closes GH-363.

Reviewed-by: Christian Murphy <[email protected]>

Co-authored-by: Sam Ruby <[email protected]>
@wooorm wooorm closed this in 42004b0 Nov 11, 2018
wooorm added a commit that referenced this pull request Nov 11, 2018
Closes GH-22.
Related to GH-20.
Related to syntax-tree/mdast#23.
Related to remarkjs/remark#346.

Co-authored-by: Sam Ruby <[email protected]>
@wooorm wooorm added ⛵️ status/released 🐛 type/bug This is a problem 🗄 area/interface This affects the public interface 🧑 semver/major This is a change labels Aug 11, 2019
@wooorm wooorm added the 💪 phase/solved Post is done label Apr 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🗄 area/interface This affects the public interface 💪 phase/solved Post is done 🧑 semver/major This is a change 🐛 type/bug This is a problem

Development

Successfully merging this pull request may close these issues.

3 participants