Skip to content

Register new snapshots - #11699

Merged
bors merged 1 commit into
rust-lang:masterfrom
alexcrichton:snapshot
Jan 21, 2014
Merged

bors merged 1 commit into
rust-lang:masterfrom
alexcrichton:snapshot

Conversation

@alexcrichton

Copy link
Copy Markdown
Member

Upgrade the version to 0.10-pre

Upgrade the version to 0.10-pre
bors added a commit that referenced this pull request Jan 21, 2014
@bors
bors merged commit cb12de1 into rust-lang:master Jan 21, 2014
@alexcrichton
alexcrichton deleted the snapshot branch January 23, 2014 18:22
flip1995 pushed a commit to flip1995/rust that referenced this pull request Nov 2, 2023
Set doc-tests to `no_run`

This excludes `should_panic` tests, those are still run to ensure they panic. Most of our other doc snippets don't gain much from being run though so this frees up a nice bit of CI time

It also fixes the occasional issue such as `foo.txt`s being created https://github.com/rust-lang/rust-clippy/blob/f942470ca774b9648bac042f5d4c4ec74b81b61a/clippy_lints/src/permissions_set_readonly_false.rs#L19

changelog: none
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
11699: feat: assist to remove unneeded `async`s r=Ethiraric a=Ethiraric

This should fix rust-lang#11392 

This PR adds a quickfix on functions marked with `async` that suggests, if and only if no `await` expression in find in the body of the function (it relies on `SyntaxNode::descendants()` to recursively traverse blocks), to remove the `async` keyword.

The lint is made so that it triggers only if the cursor is not in the body of the function, so that it does not pollute the quickfix list.

It does not trigger a diagnostic. I don't know if this repository is the place to implement those (`clippy`?). If it is, I would very much like pointers on where to start looking.

If there are test cases I haven't thought about, please do suggest.

Co-authored-by: Ethiraric <ethiraric@gmail.com>
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