Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat(markdown): full GitHub admonition#759

Open
788009 wants to merge 2 commits into
saicaca:mainfrom
788009:feat/full-github-admonition
Open

feat(markdown): full GitHub admonition#759
788009 wants to merge 2 commits into
saicaca:mainfrom
788009:feat/full-github-admonition

Conversation

@788009
Copy link
Copy Markdown

@788009 788009 commented May 11, 2026

Type of change

  • Bug fix (a non-breaking change that fixes an issue)
  • New feature (a non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Other (please describe):

Checklist

  • I have read the CONTRIBUTING document.
  • I have checked to ensure that this Pull Request is not for personal changes.
  • I have performed a self-review of my own code.
  • My changes generate no new warnings.

Changes

Currently, GitHub-style admonitions are not fully supported:

  • The IMPORTANT admonition is rendered as WARNING.
  • The CAUTION admonition is not rendered as an admonition at all.

I have made some changes as follows:

  • Bumped remark-github-admonitions-to-directives from 1.0.5 to 2.1.0 (this version supports custom admonition mapping)
  • Added the correct mapping to astro.config.mjs

How To Test

Create a post containing GitHub-style IMPORTANT or CAUTION admonitions, like

> [!IMPORTANT]
> Content

> [!CAUTION]
> Content

They should now be rendered correctly as

Important

Content

Caution

Content

Screenshots

Before

image

After

image

@vercel
Copy link
Copy Markdown

vercel Bot commented May 11, 2026

@788009 is attempting to deploy a commit to the zephyirdgmailcom's projects Team on Vercel.

A member of the Team first needs to authorize it.

@JoeyC-Dev
Copy link
Copy Markdown
Contributor

Close: #742

Same fix:
#645
#746

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.

2 participants