Skip to content

Remove automatic $ prefix for bash and console code blocks #1261

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

Merged
merged 1 commit into from
Feb 22, 2024

Conversation

Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Feb 22, 2024

This prefix is very surprising to blog authors because e.g. GitHub does not render it that way. It also does not work well if a code block contains multiple commands where all but the first one need a manual prefix. The original commit that introduced this unfortunately does not give any reasons for why it was introduced.

Related:

r? @rust-lang/website

This prefix is very surprising to blog authors because e.g. GitHub does not render it that way. It also does not work well if a code block contains multiple commands where all but the first one need a manual prefix. The original commit that introduced this unfortunately does not give any reasons for why it was introduced.
@Manishearth Manishearth merged commit 46db665 into rust-lang:master Feb 22, 2024
@kpreid
Copy link
Contributor

kpreid commented Feb 22, 2024

The original commit that introduced this unfortunately does not give any reasons for why it was introduced.

One motivation might have been to cause the $ text to not be copiable, so that someone intending to copy and run the command doesn't get the $ that is not part of the command.

@Turbo87 Turbo87 deleted the remove-console-prefix branch February 22, 2024 16:17
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.

3 participants