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

Update FIPS README.md: use new syntax for notes #1080

Merged
merged 2 commits into from
Jan 24, 2024

Conversation

dagood
Copy link
Member

@dagood dagood commented Nov 8, 2023

I saw this new syntax being used elsewhere and thought it might be interesting to try it out here.

https://docs.github.com/en/get-started/writing-on-github/getting-started-with-writing-and-formatting-on-github/basic-writing-and-formatting-syntax#alerts

[!NOTE]
Highlights information that users should take into account, even when skimming.

[!IMPORTANT]
Crucial information necessary for users to succeed.

[!WARNING]
Critical content demanding immediate user attention due to potential risks.

Note

Highlights information that users should take into account, even when skimming.

Important

Crucial information necessary for users to succeed.

Warning

Critical content demanding immediate user attention due to potential risks.

On the other hand, I think I may have overused the idea of adding notes when I was writing these sections... this feature is making it more obvious.

@dagood dagood requested a review from a team as a code owner November 8, 2023 18:42
Copy link
Member

@gdams gdams left a comment

Choose a reason for hiding this comment

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

Seems like a good formatting improvement

@dagood dagood enabled auto-merge (squash) January 24, 2024 18:19
@dagood dagood merged commit 70ff7eb into microsoft/main Jan 24, 2024
19 checks passed
@dagood dagood deleted the dev/dagood/note-syntax branch March 18, 2024 23:08
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