Skip to content

Conversation

@vinceAmstoutz
Copy link
Member

@vinceAmstoutz vinceAmstoutz commented Dec 19, 2025

Before

Screenshot 2025-12-19 at 16 28 36

Also in GitHub preview:

Screenshot 2025-12-19 at 16 28 46

After

image

So it will ok on the website.

@vinceAmstoutz
Copy link
Member Author

@copilot fix lint

Copy link
Contributor

Copilot AI commented Dec 19, 2025

@vinceAmstoutz I've opened a new pull request, #2236, to work on those changes. Once the pull request is ready, I'll request review from you.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes the formatting of a note block in the bootstrap documentation to ensure proper rendering on the website and in GitHub preview.

Key Changes:

  • Separated the note marker from its content by placing them on separate lines for proper markdown rendering

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

…separator (#2236)

* Initial plan

* fix: apply prettier formatting to bootstrap.md

Co-authored-by: vinceAmstoutz <46444652+vinceAmstoutz@users.noreply.github.com>

* fix: use correct GitHub alert syntax with empty line separator

Co-authored-by: vinceAmstoutz <46444652+vinceAmstoutz@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: vinceAmstoutz <46444652+vinceAmstoutz@users.noreply.github.com>
@vinceAmstoutz
Copy link
Member Author

vinceAmstoutz commented Dec 19, 2025

ℹ️ Since full Prettier support was restored, an extra empty line is required inside alerts (see Copilot tests in #2236).

Valid syntax for alerts before (prettier is not okay with that whereas it's the default GitHub syntax)

> [!NOTE]
> This documentation is ....

Valid syntax for alerts after (extra empty line with > is required to make prettier and rendered both OK)

> [!NOTE]
>
> This documentation is ....

I propose to keep it this way for the time being, to keep things simple.

@vinceAmstoutz vinceAmstoutz merged commit 4775684 into 4.2 Dec 19, 2025
6 checks passed
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