Skip to content

fix: support GFM protocol autolinks - #4067

Open
evgenyponomarev wants to merge 1 commit into
markedjs:masterfrom
evgenyponomarev:codex/gfm-protocol-autolinks
Open

fix: support GFM protocol autolinks#4067
evgenyponomarev wants to merge 1 commit into
markedjs:masterfrom
evgenyponomarev:codex/gfm-protocol-autolinks

Conversation

@evgenyponomarev

Copy link
Copy Markdown

Marked version: master

Markdown flavor: GitHub Flavored Markdown

Description

AI disclosure: OpenAI Codex assisted with implementation and validation; I reviewed and own the contribution.

Contributor

  • Tests exist to ensure functionality and minimize regression.
  • This is existing GFM behavior, so no feature documentation is required.

Testing

  • npm test
  • npm run rules

Committer

In most cases, this should be a different person than the contributor.

  • CI is green (no forced merge required).
  • Squash and Merge PR following conventional commit guidelines.
@vercel

vercel Bot commented Aug 21, 2026

Copy link
Copy Markdown

@evgenyponomarev is attempting to deploy a commit to the MarkedJS Team on Vercel.

A member of the Team first needs to authorize it.

@vercel

vercel Bot commented Aug 22, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
marked-website Ready Ready Preview Aug 22, 2026 4:55pm

Request Review

@UziTech

UziTech commented Aug 23, 2026

Copy link
Copy Markdown
Member

Interesting that GitHub doesn't even follow GFM 🤣

xmpp:foo@bar.baz/txt/bin

is

xmpp:foo@bar.baz/txt/bin

not

xmpp:foo@bar.baz/txt/bin

@UziTech UziTech left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks! 💯

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants