Skip to content

Conversation

@rherrmann
Copy link
Contributor

Use two consecutive newlines (\n\n) to actually continue text on a
new line.

Use proper markup to reference related settings.

Consistently format references to files, command line arguments, etc.
as code. Format mentions of UI elements in italic.

Fix typos, add missing full-stops, add missing default values.

@Veykril
Copy link
Member

Veykril commented Dec 22, 2020

The tests fail because the settings are generated from code and they check whether those are in sync now so you want to edit the lines here https://github.com/rust-analyzer/rust-analyzer/blob/ba71685343ccdca9de3899d2ae634aaa8185c359/crates/rust-analyzer/src/config.rs#L25-L174 and here instead https://github.com/rust-analyzer/rust-analyzer/blob/ba71685343ccdca9de3899d2ae634aaa8185c359/crates/rust-analyzer/src/config.rs#L714-L770
And then regenerate them(not sure how you would do this though I don't think cargo xtask codegen does that)

Use two consecutive newlines (`\n\n`) to actually continue text on a
new line.

Use proper markup to reference related settings.

Consistently format references to files, editor commands, command line
arguments, files, etc. as `code`.

Fix typos, add missing full-stops, add missing default values.
@matklad
Copy link
Contributor

matklad commented Dec 22, 2020

bors r+

Thanks!

@bors
Copy link
Contributor

bors bot commented Dec 22, 2020

@bors bors bot merged commit 013739c into rust-lang:master Dec 22, 2020
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