Skip to content

Update usage.md - #1520

Closed
kstefanini wants to merge 2 commits into
dustin10:masterfrom
kstefanini:patch-1
Closed

Update usage.md#1520
kstefanini wants to merge 2 commits into
dustin10:masterfrom
kstefanini:patch-1

Conversation

@kstefanini

Copy link
Copy Markdown

Hey!
I just ran into difficulties to make things work so I propose to move information to a more readable place

Move doctrine condition somewhere its more difficult to miss
@garak

garak commented Jul 29, 2025

Copy link
Copy Markdown
Collaborator

There's already an "important" block below.
Instead of adding a redundant note, we should enhance that with the proper markdown, e.g. [!IMPORTANT] instead of **Important:**.
By the way, all the blocks should be enhanced in the same way

@kstefanini

Copy link
Copy Markdown
Author

Thanks for your quick reply, I've updated my proposal

@garak

garak commented Jul 29, 2025

Copy link
Copy Markdown
Collaborator

At this point, why not update all the other pages as well?

@garak

garak commented Aug 21, 2025

Copy link
Copy Markdown
Collaborator

If you rebase, the CS failures should go.
After that, if you could please apply the suggested changes, I'll be glad to merge this

@garak garak closed this Feb 25, 2026
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