Skip to content
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

Add isAutolink property to Link #110

Merged

Conversation

Kyle-Ye
Copy link
Contributor

@Kyle-Ye Kyle-Ye commented Mar 13, 2023

Bug/issue #, if applicable:

With swiftlang/swift-docc#376

Close swiftlang/swift-docc#271

Summary

  • Add a new property isAutolink to Link
  • Add a setter to Link.isAutolink so that after updating destination of link, we can preserve the old isAutolink value.

See discussion on swiftlang/swift-docc#376 (comment)

Dependencies

None

Testing

swiftlang/swift#61506

Checklist

Make sure you check off the following items. If they cannot be completed, provide a reason.

  • Added tests
  • Ran the ./bin/test script and it succeeded
  • Updated documentation if necessary

@Kyle-Ye Kyle-Ye requested a review from franklinsch March 13, 2023 12:00
@Kyle-Ye Kyle-Ye force-pushed the bugfix/kyle/custom-link-titles branch from d5f82a2 to ac1596e Compare March 13, 2023 12:01
@Kyle-Ye
Copy link
Contributor Author

Kyle-Ye commented Mar 13, 2023

@swift-ci please test

@Kyle-Ye Kyle-Ye requested a review from QuietMisdreavus March 13, 2023 12:07
@Kyle-Ye Kyle-Ye force-pushed the bugfix/kyle/custom-link-titles branch from ac1596e to 2c83cf7 Compare March 13, 2023 12:09
@Kyle-Ye
Copy link
Contributor Author

Kyle-Ye commented Mar 13, 2023

@swift-ci please test

@Kyle-Ye Kyle-Ye force-pushed the bugfix/kyle/custom-link-titles branch from e24e7c6 to df6944a Compare March 13, 2023 15:00
@Kyle-Ye Kyle-Ye requested a review from QuietMisdreavus March 13, 2023 15:01
@Kyle-Ye
Copy link
Contributor Author

Kyle-Ye commented Mar 13, 2023

@swift-ci please test

Copy link
Contributor

@QuietMisdreavus QuietMisdreavus left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@QuietMisdreavus QuietMisdreavus merged commit 656cf39 into swiftlang:main Mar 13, 2023
QuietMisdreavus added a commit that referenced this pull request Mar 13, 2023
@Kyle-Ye Kyle-Ye deleted the bugfix/kyle/custom-link-titles branch March 14, 2023 11:06
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.

Custom link titles authored with []() syntax for doc links are silently ignored
2 participants