Skip to content

docs: Remove $ sign in code examples#454

Closed
Swiftaxe wants to merge 1 commit into
mainfrom
feat/remove_dollar_symbol
Closed

docs: Remove $ sign in code examples#454
Swiftaxe wants to merge 1 commit into
mainfrom
feat/remove_dollar_symbol

Conversation

@Swiftaxe
Copy link
Copy Markdown
Collaborator

@Swiftaxe Swiftaxe commented Mar 31, 2026

Addresses the dollar signs in code examples and closes #515

@Swiftaxe Swiftaxe force-pushed the feat/remove_dollar_symbol branch from 0574e4e to 0dccdae Compare April 1, 2026 08:01
Copy link
Copy Markdown
Collaborator

@marcelomendoncasoares marcelomendoncasoares left a comment

Choose a reason for hiding this comment

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

Just one question on change that seems unrelated.

Also, should we remove the $ on past documentations (the versioned_docs folder) as well?

Comment thread docs/index.md Outdated
@marcelomendoncasoares
Copy link
Copy Markdown
Collaborator

Also one more $ from #438.

@marcelomendoncasoares
Copy link
Copy Markdown
Collaborator

Is this still needed after #480?

@Swiftaxe Swiftaxe force-pushed the feat/remove_dollar_symbol branch from 0dccdae to dbc0125 Compare May 19, 2026 05:48
@Swiftaxe Swiftaxe requested a review from Copilot May 19, 2026 05:49
Copy link
Copy Markdown
Contributor

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.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@Swiftaxe
Copy link
Copy Markdown
Collaborator Author

Is this still needed after #480?

You are right! Will close this, and the issue.

@Swiftaxe Swiftaxe closed this May 19, 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.

Remove the $ symbol in all md to get copy & paste more easy

3 participants