Skip to content

Conversation

@hush-hush
Copy link
Member

What does this PR do? What is the motivation?

Remove '$' since one is automatically added when showing bash script. Adding one manually produce double dollar signs.

Merge instructions

None

Merge readiness:

  • Ready for merge

@hush-hush hush-hush requested a review from a team as a code owner June 6, 2025 16:28
@github-actions
Copy link
Contributor

github-actions bot commented Jun 6, 2025

✅ Documentation Team Review

The documentation team has approved this pull request. Thank you for your contribution!

@github-actions
Copy link
Contributor

github-actions bot commented Jun 6, 2025

Preview links (active after the build_preview check completes)

Modified Files

@jhgilbert
Copy link
Contributor

Hi @hush-hush, based on the build preview, this change doesn't seem to have accomplished the goal of automatically adding a $ to the commands. Did you intend to use a code-block shortcode?

Copy link
Contributor

@hestonhoffman hestonhoffman left a comment

Choose a reason for hiding this comment

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

This should do it

@hush-hush hush-hush force-pushed the maxime/fix-typo-secret-doc branch from 665e991 to fe558af Compare June 17, 2025 13:06
@hush-hush hush-hush requested a review from hestonhoffman June 18, 2025 14:24
@hush-hush hush-hush merged commit 6a59b99 into master Jun 18, 2025
20 of 21 checks passed
@hush-hush hush-hush deleted the maxime/fix-typo-secret-doc branch June 18, 2025 16:27
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.

4 participants