Skip to content

[Site] refactor commonmark config #1791

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

Merged
merged 1 commit into from
Apr 26, 2024
Merged

Conversation

kbond
Copy link
Member

@kbond kbond commented Apr 26, 2024

Q A
Bug fix? no
New feature? no
Issues n/a
License MIT

This change allows us to add commonmark configuration to the extensions. Also, I refactored the Changelog to use native commonmark extension for github mentions/issue links (fixes a few bugs too).

@carsonbot carsonbot added the Site ux.symfony.com label Apr 26, 2024
@kbond kbond requested a review from smnandre April 26, 2024 15:42
@carsonbot carsonbot added the Status: Needs Review Needs to be reviewed label Apr 26, 2024
@kbond kbond force-pushed the site/commonmark-refactor branch from a883b9f to 9e6cedb Compare April 26, 2024 15:47
@kbond kbond force-pushed the site/commonmark-refactor branch from 9e6cedb to 4493319 Compare April 26, 2024 15:53
@smnandre
Copy link
Member

Also, I refactored the Changelog to use native commonmark extension for github mentions/

Wow !

fixes a few bugs too

❤️

@carsonbot carsonbot added Status: Reviewed Has been reviewed by a maintainer and removed Status: Needs Review Needs to be reviewed labels Apr 26, 2024
@kbond kbond merged commit 32c222b into symfony:2.x Apr 26, 2024
1 of 2 checks passed
@kbond kbond deleted the site/commonmark-refactor branch April 26, 2024 16:54
]);

$converter->getEnvironment()
->addRenderer(FencedCode::class, new CodeRenderer(), 10)
Copy link
Member

Choose a reason for hiding this comment

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

Soon Tempest's highlighting 👀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Site ux.symfony.com Status: Reviewed Has been reviewed by a maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants