fix: delimit parenthesized autolinks - #291
Conversation
|
@alecdotdev This is ready for review after #286. It adds a regression test and keeps the closing parenthesis plus adjacent text out of the bare URL. |
|
@alecdotdev GitHub recently introduced Stacked Pull Requests in private preview: https://github.github.com/gh-stack/ I have several follow-up PRs with genuine commit ancestry. Today they all target I would not convert unrelated PRs into one long stack; I would use it only for small, clearly dependent sequences. If you prefer the current workflow, I will continue to rebase and submit each PR independently. |
|
@PathGao I've been looking forward to stacked PRs. I'll need to transfer the repo to an organization so I'll look into transferring that asap. |
|
@alecdotdev #291 is now clean and ready to merge. I have also rebased my open PRs from #310 onward onto the current When you have time, could you please share maintainer direction on #209, #230, and #281? The decisions there affect the macOS release/signing path, embedded-video handling, and application-menu/settings work that I plan to take on next. Thank you. |
|
@PathGao transferred to org, according to GitHub, stacked PRs should now work. not sure if the |
Fixes #290
Parenthesized bare URLs followed immediately by text are rendered as explicit Markdown links, so the closing parenthesis and following text stay outside the URL.
Includes a Rust regression test for the reported Speedtest URL.
Depends on #286 because upstream master currently has the toolbar-test regression fixed there; merge #286 first.