Skip to content
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

doc: Fix the markdown highlighting syntax #5539

Merged
merged 1 commit into from Nov 15, 2023

Conversation

YDX-2147483647
Copy link
Contributor

@YDX-2147483647 YDX-2147483647 commented Nov 15, 2023

There should be a new line after [!WARNING], according to community · Discussion #16925.

Before

[!WARNING] uutils is original code and cannot contain any code from GNU or
other implementations. This means that we cannot accept any changes based on
the GNU source code
. To make sure that cannot happen, you cannot link to
the GNU source code
either.

After

Warning

uutils is original code and cannot contain any code from GNU or
other implementations. This means that we cannot accept any changes based on
the GNU source code
. To make sure that cannot happen, you cannot link to
the GNU source code
either.

There should be a new line after `[!WARNING]`, according to [community · Discussion #16925](https://github.com/orgs/community/discussions/16925).
@cakebaker cakebaker merged commit c8b8509 into uutils:main Nov 15, 2023
44 of 52 checks passed
@cakebaker
Copy link
Contributor

Good catch, thanks!

Copy link

GNU testsuite comparison:

GNU test failed: tests/tail/retry. tests/tail/retry is passing on 'main'. Maybe you have to rebase?
GNU test failed: tests/tail/symlink. tests/tail/symlink is passing on 'main'. Maybe you have to rebase?
GNU test failed: tests/tail/truncate. tests/tail/truncate is passing on 'main'. Maybe you have to rebase?

@YDX-2147483647 YDX-2147483647 deleted the patch-1 branch November 15, 2023 14:04
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.

None yet

2 participants