-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[Markup] add support for swift-cmark's inline attributes #60173
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
Conversation
the older names were kept in the cmark header for compatibility, but these are the ones that are actually defined in the main enum
rdar://96830173
@swift-ci Please test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just fantastic. I only have one suggestion for the name of the element to make it a little easier to navigate to in the future.
@swift-ci Please test |
@swift-ci Please test |
bindings/xml/comment-xml-schema.rng
Outdated
@@ -959,6 +959,13 @@ | |||
<param name="pattern">.*\S.*</param> | |||
</data> | |||
</element> | |||
<element name="Attribute"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry I missed this. It looks like you forgot to update the schema and XML printing.
@swift-ci Please test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, everything looks great.
Resolves rdar://96830173
When the swift-cmark branch was changes from
main
togfm
in #40188, swiftMarkup was not updated to handle the new kinds of nodes that were introduced. This causes the compiler to crash when using swift-cmark's "inline attributes" (or Foundation's attributed strings, which use the same syntax). This PR updates swiftMarkup to handle inline attributes from swift-cmark.