prefix for bash scripts in doc by xtexx · Pull Request #99845 · rust-lang/rust · GitHub","description":"root tab size preference pre, code tab size var tab size preference locale featureFlags actions custom images storage billing visibility actions image versio...","inLanguage":"en-US","isPartOf":{"@id":"https://kelaenderkoe.pages.dev/#website"},"datePublished":"2026-04-09T16:02:45.243Z","dateModified":"2026-04-09T16:02:45.243Z","breadcrumb":{"@id":"https://kelaenderkoe.pages.dev/float-https-github.com/rust-lang/rust/pull/99845#breadcrumb"},"potentialAction":[{"@type":"ReadAction","target":["https://kelaenderkoe.pages.dev/float-https-github.com/rust-lang/rust/pull/99845"]}]},{"@type":"Article","@id":"https://kelaenderkoe.pages.dev/float-https-github.com/rust-lang/rust/pull/99845#article","headline":"Remove ` Remove `$` prefix for bash scripts in doc by xtexx · Pull Request #99845 · rust-lang/rust · GitHub Remove `$` prefix for bash scripts in doc by xtexx · Pull Request #99845 · rust-lang/rust · GitHub Remove `$` prefix for bash scripts in doc by xtexx · Pull Request #99845 · rust-lang/rust · GitHub Remove `$` prefix for bash scripts in doc by xtexx · Pull Request #99845 · rust-lang/rust · GitHub Remove `$` prefix for bash scripts in doc by xtexx · Pull Request #99845 · rust-lang/rust · GitHub prefix for bash scripts in doc by xtexx · Pull Request #99845 · rust-lang/rust · GitHub","description":"root tab size preference pre, code tab size var tab size preference locale featureFlags actions custom images storage billing visibility actions image versio...","url":"https://kelaenderkoe.pages.dev/float-https-github.com/rust-lang/rust/pull/99845","datePublished":"2026-04-09T16:02:45.243Z","dateModified":"2026-04-09T16:02:45.243Z","author":{"@type":"Person","name":"Admin","url":"https://kelaenderkoe.pages.dev"},"publisher":{"@type":"Organization","name":"BERJAYA","url":"https://kelaenderkoe.pages.dev","logo":{"@type":"ImageObject","url":"https://kelaenderkoe.pages.dev/logo.png","width":600,"height":60}},"image":{"@type":"ImageObject","url":"https://avatars.githubusercontent.com/u/46394906?s=80&v=4","width":1200,"height":630},"mainEntityOfPage":{"@type":"WebPage","@id":"https://kelaenderkoe.pages.dev/float-https-github.com/rust-lang/rust/pull/99845"},"inLanguage":"en-US","keywords":"copilot, rust, lang, pull, false, font, this, code, copy, 2022, 99845, github, request, loading, bors"},{"@type":"WebSite","@id":"https://kelaenderkoe.pages.dev/#website","url":"https://kelaenderkoe.pages.dev","name":"BERJAYA","description":"Latest news, articles and resources from BERJAYA","publisher":{"@type":"Organization","name":"BERJAYA","logo":{"@type":"ImageObject","url":"https://kelaenderkoe.pages.dev/logo.png","width":600,"height":60}},"inLanguage":"en-US","potentialAction":[{"@type":"SearchAction","target":{"@type":"EntryPoint","urlTemplate":"https://kelaenderkoe.pages.dev/?s={search_term_string}"},"query-input":"required name=search_term_string"}]},{"@type":"BreadcrumbList","@id":"https://kelaenderkoe.pages.dev/float-https-github.com/rust-lang/rust/pull/99845#breadcrumb","itemListElement":[{"@type":"ListItem","position":1,"name":"Home","item":"https://kelaenderkoe.pages.dev"},{"@type":"ListItem","position":2,"name":"Remove ` Remove `$` prefix for bash scripts in doc by xtexx · Pull Request #99845 · rust-lang/rust · GitHub Remove `$` prefix for bash scripts in doc by xtexx · Pull Request #99845 · rust-lang/rust · GitHub Remove `$` prefix for bash scripts in doc by xtexx · Pull Request #99845 · rust-lang/rust · GitHub Remove `$` prefix for bash scripts in doc by xtexx · Pull Request #99845 · rust-lang/rust · GitHub Remove `$` prefix for bash scripts in doc by xtexx · Pull Request #99845 · rust-lang/rust · GitHub prefix for bash scripts in doc by xtexx · Pull Request #99845 · rust-lang/rust · GitHub","item":"https://kelaenderkoe.pages.dev/float-https-github.com/rust-lang/rust/pull/99845"}]}]}
Skip to content

Remove $ prefix for bash scripts in doc#99845

Merged
bors merged 1 commit intorust-lang:masterfrom
xtexx:patch-1
Jul 29, 2022
Merged

Remove $ prefix for bash scripts in doc#99845
bors merged 1 commit intorust-lang:masterfrom
xtexx:patch-1

Conversation

@xtexx
Copy link
Copy Markdown
Contributor

@xtexx xtexx commented Jul 28, 2022

No description provided.

@rust-highfive
Copy link
Copy Markdown
Contributor

rust-highfive commented Jul 28, 2022

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @GuillaumeGomez (or someone else) soon.

Please see the contribution instructions for more information.

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 28, 2022
@GuillaumeGomez
Copy link
Copy Markdown
Member

GuillaumeGomez commented Jul 28, 2022

@bors r+ rollup

@bors
Copy link
Copy Markdown
Collaborator

bors commented Jul 28, 2022

📌 Commit 957fe0b has been approved by GuillaumeGomez

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 28, 2022
@xtexx
Copy link
Copy Markdown
Contributor Author

xtexx commented Jul 28, 2022 •
edited
Loading

I noticed that lots of bash scripts has the $ prefix, so may be it is correct?

However, at instrument-coverage.md#L49, the script does not have the prefix.

Which is right? Should I add all of them or remove all of them?

@JohnTitor JohnTitor mentioned this pull request Jul 29, 2022
Merged
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 29, 2022
@bors
Auto merge of rust-lang#99892 - JohnTitor:rollup-qi4fem8, r=JohnTitor
2f847b8 
Rollup of 8 pull requests

Successful merges:

 - rust-lang#99686 (add suggestion when there is a impl of external trait on pointer with wrong coherence rules)
 - rust-lang#99760 (doc/rustc: describe the uefi target platforms)
 - rust-lang#99766 (Htmldocck: Substitute the doc channel when blessing)
 - rust-lang#99781 (Use String::from_utf8_lossy in CStr demo)
 - rust-lang#99803 (Update mentions to `rustc_metadata::rmeta::Lazy`)
 - rust-lang#99845 (Remove `$` prefix for bash scripts in doc)
 - rust-lang#99850 (rustdoc: Remove more Clean trait implementations)
 - rust-lang#99872 (Clone the `src/llvm-project` submodule if profiling is enabled)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit da3f951 into rust-lang:master Jul 29, 2022
@rustbot rustbot added this to the 1.64.0 milestone Jul 29, 2022
@xtexx
Copy link
Copy Markdown
Contributor Author

xtexx commented Jul 29, 2022

@GuillaumeGomez request for review, see the last comment for more.

@GuillaumeGomez
Copy link
Copy Markdown
Member

GuillaumeGomez commented Jul 29, 2022

It's a visual marker to see the different commands. Whether or not it's there doesn't matter much.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@GuillaumeGomez GuillaumeGomez GuillaumeGomez approved these changes

Assignees

@GuillaumeGomez GuillaumeGomez

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Milestone

1.64.0

Development

Successfully merging this pull request may close these issues.

5 participants

@xtexx @rust-highfive @GuillaumeGomez @bors @rustbot