Add `dev deprecate` to the development basics

Auto merge of #9074 - daxpedda:equatable-if-let-external-macro, r=Manishearth

Fix false-positive in `equatable_if_let`

Was linting in external macros. I guess now that I know about https://github.com/rust-lang/rust-clippy/pull/8694 it seems all kinda pointless until we resolve that.

Nevertheless, it's an improvement.

Fixes #9066.

changelog:`equatable_if_let` No longer lint on macros

Fix false-positive in `equatable_if_let`

Auto merge of #9071 - Alexendoo:8734, r=dswij

Uncomment test for #8734

I believe the issue was an interaction between rustfix and `span_lint_and_sugg_for_edges`, so this would've been fixed by https://github.com/rust-lang/rust/pull/98261 (Thanks, `@WaffleLapkin!)`

Closes #8734

changelog: none

Auto merge of #9070 - flip1995:ci-fix, r=xFrednet

Make sure bors success depends on metadata_collection

r? `@xFrednet`

Currently bors runs the `metadata_collection` but merges before the run is finished, because the bors success dummy step didn't depend on it. This also makes sure that the `metadata_collection` test is run at the same time as the other base runs to not produce overhead.

changelog: none

Uncomment test for #8734

Make sure bors success depends on metadata_collection

Auto merge of #9069 - flip1995:rustup, r=flip1995

Rustup

r? `@ghost`

changelog: none

Bump Clippy version -> 0.1.64

Bump nightly version -> 2022-06-30

Fix dogfood

Merge remote-tracking branch 'upstream/master' into rustup

Auto merge of #9068 - tamaroning:fix-typo-2022-6-30, r=giraffate,xFrednet

fix typo in Clippy Lints page

https://rust-lang.github.io/rust-clippy/master/#significant_drop_in_scrutinee

changelog: none

Update large_const_arrays.rs

Auto merge of #8666 - Jarcho:while_let_loop_7913, r=dswij

Don't lint `while_let_loop` when significant drop order would change

fixes #7226
fixes #7913
fixes #5717

For #5717 it may not stay fully fixed. This is only completely fixed right now due to all the allowed drop impls have `#[may_dangle]` on their drop impls. This may get changed in the future based on how significant drops are determined, but the example listed with `RefCell` shouldn't break.

changelog: Don't lint `while_let_loop` when the order of significant drops would change

fix

Don't lint `while_let_loop` when drop order would change

Extract util functions from `redundant_pattern_match`

Auto merge of #9043 - dswij:9037-test, r=Manishearth

`extra_unused_lifetimes` add FP test case emitting from derived attributes.

Add test to cover for #9014 which is fixed in #9037.

changelog: [`extra_unused_lifetimes`] Add FP test case emitting from derived attributes.

---

Seeing the FP from the test:

```sh
$ git revert -m 1 1d1ae10876d3eaa5c982dd3daa083b7c2fc363b9
$ TESTNAME=extra_unused_lifetime cargo uitest
```

Auto merge of #8981 - PrestonFrom:more_details_for_significant_drop_lint, r=flip1995

Add details about how significant drop in match scrutinees can cause deadlocks

Adds more details about how a significant drop in a match scrutinee can cause a deadlock and include link to documentation.

changelog: Add more details to significant drop lint to explicitly show how temporaries in match scrutinees can cause deadlocks.

Auto merge of #9046 - xFrednet:rust-97660-expection-something-something, r=Jarcho

Fix `#[expect]` for most clippy lints

This PR fixes most `#[expect]` - lint interactions listed in rust-lang/rust#97660. [My comment in the issue](https://github.com/rust-lang/rust/issues/97660#issuecomment-1147269504) shows the current progress (Once this is merged). I plan to work on `duplicate_mod` and `multiple_inherent_impl` and leave the rest for later. I feel like stabilizing the feature is more important than fixing the last few nits, which currently also don't work with `#[allow]`.

---

changelog: none

r? `@Jarcho`

cc: rust-lang/rust#97660

Auto merge of #8355 - Jarcho:explicit_auto_deref_2, r=flip1995

Add lint `explicit_auto_deref` take 2

fixes: #234
fixes: #8367
fixes: #8380

Still things to do:

* ~~This currently only lints `&*<expr>` when it doesn't trigger `needless_borrow`.~~
* ~~This requires a borrow after a deref to trigger. So `*<expr>` changing `&&T` to `&T` won't be caught.~~
* The `deref` and `deref_mut` trait methods aren't linted.
* Neither ~~field accesses~~, nor method receivers are linted.
* ~~This probably shouldn't lint reborrowing.~~
* Full slicing to deref should probably be handled here as well. e.g. `&vec[..]` when just `&vec` would do

changelog: new lint `explicit_auto_deref`

Auto merge of #8921 - Jarcho:merge_passes, r=llogiq

Add `Operators` lint pass

changelog: None

Move `SelfAssignment` into `Operators` lint pass

Fix dogfood

Move `PtrEq` into `Operators` lin pass

Move `NeedlessBitwiseBool` into `Operators` lint pass

Move `ModuloArithmetic` into `Operators` lint pass

Move some lints from `Misc` to `Operators`

Move `IntegerDivision` into `Operators` lint pass

Move `IdentityOp` into `Operators` lint pass

Move `FloatEqualityWithoutAbs` into `Operators` lint pass

Move `ErasingOp` into `Operators` lint pass

Move `EqOp` into `Operators` lint pass

Move `DurationSubsec` into `Operators` lint pass

Move `DoubleComparison` into `Operators` lint pass

Move `BitMask` into `Operators` lint pass

Move `AssignOps` into `Operators` lint pass

Move `Arithmetic` into `Operators` lint pass

Add `Operators` lint pass

Add `explicit_auto_deref` test for variadic function

Code cleanup

Handle future precedence issues in `explicit_auto_deref` + cleanup

Fix `needless_borrow` suggestion when calling a trait method taking `self`

Refactor `dereference.rs`

Merge `Position` and `AutoDerefStability`

Lint `explicit_auto_deref` immediately after `needless_borrow`

Lint `explicit_auto_deref` without a leading borrow

Only check parent node once in `dereference.rs`

Lint field accesses in `explicit_auto_deref`

Merge different parent walking loops in `dereference.rs`

`needless_borrow` will now walk further to find the target type.

Don't lint `explicit_auto_deref` on reborrows

Add `explicit_auto_deref` lint

Better support projection types when finding the signature for an expression

Update `boxed_local` expect attribute location

Auto merge of #8774 - hellow554:cargo-rust-version, r=flip1995

try reading rust-version from Cargo.toml

Cargo.toml can contain a field `rust-version`, that acts like a MSRV of
clippy.toml file: https://doc.rust-lang.org/cargo/reference/manifest.html#the-rust-version-field
This will try to read that field and use it, if the clippy.toml config
has no `msrv` entry

changelog: respect `rust-version` from `Cargo.toml`

closes #8746
closes #7765

Auto merge of #8639 - Jarcho:trivially_copy_pass_by_ref_5953, r=dswij

`trivially_copy_pass_by_ref` fixes

fixes #5953
fixes #2961

The fix for #5953 is overly aggressive, but the suggestion is so bad that it's worth the false negatives. Basically three things together:
* It's not obviously wrong
* It compiles
* It may actually work when tested

changelog: Don't lint `trivially_copy_pass_by_ref` when unsafe pointers are used.
changelog: Better track lifetimes when linting `trivially_copy_pass_by_ref`.

add uitests for cargo rust-version field

parse `Cargo.toml` file in ui-cargo tests

compiletest_rs is not meant to test full cargo projects, but instead
only files.
So we need to parse the `Cargo.toml` file ourself and set the
corresponding environment variable. In this case we just set
`CARGO_PKG_RUST_VERSION`, nothing more. But, of course, this can be
extended.

update README.md to reflect rust-version in cargo.toml

try reading rust-version from Cargo.toml

Cargo.toml can contain a field `rust-version`, that acts like a MSRV of
clippy.toml file: https://doc.rust-lang.org/cargo/reference/manifest.html#the-rust-version-field
This will try to read that field and use it, if the clippy.toml config
has no `msrv` entry

Run cargo dev fmt

Make messages more accurate, check lint enabled

Auto merge of #9045 - alex-semenyuk:self_assignment_example, r=llogiq

Fix example `SELF_ASSIGNMENT`

changelog: Fix example in `SELF_ASSIGNMENT` docs

Auto merge of #8649 - ebobrow:imperative_find, r=flip1995

add [`manual_find`] lint for function return case

part of the implementation discussed in #7143

changelog: add [`manual_find`] lint for function return case

Don't lint `trivially_copy_pass_by_ref` when unsafe pointers are used

Actually check lifetimes in `trivially_copy_pass_by_ref`

Fix example `SELF_ASSIGNMENT`

Auto merge of #8972 - kyoto7250:use_retain, r=llogiq

feat(new lint): new lint `manual_retain`

close #8097

This PR is  a new  lint implementation.
This lint checks if the `retain` method is available.

Thank you in advance.

changelog: add new ``[`manual_retain`]`` lint

Auto merge of #8990 - tsoutsman:master, r=llogiq

Fix `let_undescore_lock` false-positive when binding without locking

Fixes #8486.

changelog: Fix `let_undescore_lock` false-positive when binding without locking.

Fix `let_undescore_lock` false-positive when binding without locking

Signed-off-by: Klim Tsoutsman <klimusha@gmail.com>

Auto merge of #9054 - alex-semenyuk:string_add_example, r=giraffate

STRING_ADD example

changelog: none
STRING_ADD example, how it should be

Auto merge of #8871 - Serial-ATA:cargo-dev-deprecate, r=giraffate

Add `cargo dev deprecate`

changelog: none

I wrote this awhile ago when `regex` was still a dependency. Is it alright to add it back?

Auto merge of #8939 - Jarcho:transmute_ptr_to_ref_lt, r=giraffate

Suggest `pointer::cast` when possible in `transmute_ptr_to_ref`

fixes #8924

changelog: Suggest casting the pointer for any type containing lifetimes in `transmute_ptr_to_ref`.
changelog: Suggest `pointer::cast` when possible in `transmute_ptr_to_ref`.

cargo dev update_lints

fix for git rebase

rename use_retain => manual_retain

check msrv

rewrite without if_chain macro

check  method

change lint type from style to perf

remove needless return

Update clippy_lints/src/use_retain.rs

Co-authored-by: llogiq <bogusandre@gmail.com>

feat(new lint): new lint `use_retain`

Auto merge of #9032 - kyoto7250:issue_9018, r=llogiq

enum_variant_names should ignore when all prefixes are _

close #9018

When Enum prefix is only an underscore, we should not issue warnings.

changelog: fix false positive in enum_variant_names

Auto merge of #9055 - kyoto7250:update_nodejs_in_github_action, r=Manishearth

update node.js version in `remark.yml`

Optional chain (`?.`) is available in `node v14`, but node version in CI is `node v12`, so CI is failed now.

ref:
https://github.com/rust-lang/rust-clippy/runs/7059529735?check_suite_focus=true

optional chain:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Optional_chaining#browser_compatibility

Corresponding PR (maybe)
https://github.com/remarkjs/remark/pull/1007

changelog: None

update node.js version

STRING_ADD example

STRING_ADD example

use all instead of join

treat argument as a slice.

Co-authored-by: llogiq <bogusandre@gmail.com>

Auto merge of #8985 - botahamec:single-match-option, r=llogiq

Lint `[single_match]` on `Option` matches

fixes #8928

changelog: did some cleanup of the logic for ``[`single_match`]`` and ``[`single_match_else`]`` which fixes the bug where `Option` matches were not linted unless a wildcard was used for one of the arms.

Give clippy some Dogfood to make it happy =^.^=

Add note to `is_lint_allowed` about lint emission

Fix `#[expect]` for `clippy::manual_non_exhaustive`

Update `useless_attribute` docs and allow `macro_use_imports`

Fix `#[expect]` for `clippy::macro_use_imports`

Fix `#[expect]` for `unnecessary_unwrap`, `panicking_unwrap`

Fix `#[expect]` for `clippy::needless_return`

Fix `#[expect]` for `clippy::implicit_return`

Fix `#[expect]` for `clippy::boxed_local`