Auto merge of #5165 - matthiaskrgr:ci_cargo_cache_autoclean, r=flip1995

gha: install cargo-cache with ci-autoclean feature which speeds up build time by a large margin.

changelog: none

gha: install cargo-cache with ci-autoclean feature which speeds up build time by a large margin.

Auto merge of #5164 - flip1995:gha_deploy_fix, r=flip1995

Fix tag deployment on GHA

I just want to test some things here currently.

changelog: none

Run deploy workflow on tags, instead of releases

Auto merge of #5162 - flip1995:gha_fix, r=flip1995

Fix GHA after the merge

changelog: none

Use list instead of array for branch names

This is the officially supported method

Auto merge of #5088 - rust-lang:gha, r=Manishearth,flip1995

Switch to GitHub Actions - Part 2 - From within

This is a continuation of #5071. This time from a branch inside the rust-lang/rust-clippy repo, not from my fork, since secrets are not available in PRs from forks.

Copying the description of #5071 to here:

Closes #4577

~~This is just an experiment. I don't think we have a consensus _if_ we should move away from travis/appveyor.~~ We have consensus: https://github.com/rust-lang/rust-clippy/pull/5071#issuecomment-576647638

~~GHA would let us run up to 20 concurrent jobs. Since we have 15 integration tests and 4 (linux, linux 32-bit, macos, windows) basic tests, we would be able to run everything concurrently.~~ The org has a limit of 60 jobs across the org, so we limit the matrix of the integration tests to 6 concurrent jobs.

~~Also IIUC we only have to build Clippy once for every initegration test and then only check the repos.~~ Nope, dependent jobs exist, but they won't keep the artifacts (not even the checkout).

TODO before merge:

- [x] Add `DEPLOY_KEY` secret to github repo
- [x] test deployment on test branch `gh-test`#
  - [x] Test normal deployment
  - [x] Test deployment no changes
  - [x] Test deployment of tag
- [x] talk with `@rust-lang/infra` for bors, `@rust-lang/infra` is good with the move (crater also uses GHA+bors)
- [x] ~~Get remark + clippy_dev check to work on pushes (https://github.community/t5/GitHub-Actions/~Builds-are-not-triggered-with-on-paths/m-p/44075; I contacted GH support about this) ~~That seems to start working again yesterday. Let's hope it keeps working.~~ Or not: df9be48. Now it works again: 723786a. I think we should wait, until this is reliable. It appears, that it doesn't work on force pushes (sometimes?): 5814142~~ We need to run the bors tests unconditionally anyway (47138d1) so it doesn't really matter.
- [x] ~~impl homu checks for GHA https://github.com/rust-lang/rust-clippy/pull/5071#issuecomment-576642983 -- I prepared: https://github.com/flip1995/rust-central-station/commit/f40230dc3cebfe4120c9dce027f61c18dd8935cb. I'd suggest to first add GHA and keep the travis and appveyor checks for a few days and to remove them in a second pass. The bors dummy jobs are added in https://github.com/rust-lang/rust-clippy/pull/5088/commits/1a83b7ad7a45aa8db4db7da55a45931e725a1ca3 and work as expected: https://github.com/rust-lang/rust-clippy/pull/5088#issuecomment-582055695. I opened https://github.com/rust-lang/rust-central-station/pull/578~~ See https://github.com/rust-lang/rust-clippy/pull/5088#issuecomment-584581420
- [x] ~Add GHA badge to Cargo.toml (blocked on rust-lang/crates.io # 1838)~ Added a FIXME in
2332b57
- [x] ~Maybe we should also wait until GHA supports yaml anchors. https://github.community/t5/GitHub-Actions/Support-for-YAML-anchors/td-p/30336/~ WIll probably not be implemented in the near future.
- [x] Add back travis + appveyor files for transition period (!)

changelog: none

Don't upload target/release in integration_build

This dir was generated by compiling RTIM, when no cache is available

Use nightly cargo to install cargo-cache

Stable cargo install still errors when packages exists, instead of updating it

Install all required remark packages in travis

Remove deployment from travis

Apply review comments

Run bors checks unconditionally

This is important, since bors doesn't know optional jobs. Otherwise bors
would time out, when a check doesn't get run.

Add bors dummy jobs

Use hash of Cargo.lock file in cache name

Add two more error causes to integration test

Build Clippy before running integration tests

Allow non-ascii characters in changelog check

Extract deployment

Extract driver test

Sort output of export.py script

Limit parallel jobs on try/r+ runs

Rate limit of 60 applies org wide, let's not spam.
Thanks pietroalbini for the hint

Cancle builds on new commits

Add GHA badge to README.md

Add a FIXME to Cargo.toml

Small change to comment in .gitignore

Add deployment to GHA

Move integration tests in clippy_bors.yml

Some call it the integration integration

Add Clippy Test to GHA

Get {RUSTUP,MULTIRUST}_{HOME,TOOLCHAIN} from runtime environment

Keep the fallback to compile-time environment

Check if changelog exists

Ignore fmt test in GHA CI

This is already checked by clippy_dev.yml GHA

Add Clippy Test (bors) to GHA

Fix markdown Files

Rendered:

[CHANGELOG.md](https://github.com/flip1995/rust-clippy/blob/gha/CHANGELOG.md)
[CODE_OF_CONDUCT.md](https://github.com/flip1995/rust-clippy/blob/gha/CODE_OF_CONDUCT.md)
[CONTRIBUTING.md](https://github.com/flip1995/rust-clippy/blob/gha/CONTRIBUTING.md)
[README.md](https://github.com/flip1995/rust-clippy/blob/gha/README.md)
[clippy_dummy/PUBLISH.md](https://github.com/flip1995/rust-clippy/blob/gha/clippy_dummy/PUBLISH.md)
[doc/adding_lints.md](https://github.com/flip1995/rust-clippy/blob/gha/doc/adding_lints.md)
[etc/relicense/RELICENSE_DOCUMENTATION.md](https://github.com/flip1995/rust-clippy/blob/gha/etc/relicense/RELICENSE_DOCUMENTATION.md)

Rename .remarkrc.json -> .remarkrc and fix file

Add remark check to GHA

Add clippy_dev checks to GitHub Actions

Add integration test GitHub action

Auto merge of #5158 - lzutao:callbackshook, r=flip1995

Impl DefaultCallbacks on our side

cc https://github.com/rust-lang/rust/pull/69059#discussion_r377643560

changelog: none

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

Rustup to rust-lang/rust#68725

Preparation for rust-lang/rust#68725

changelog: none

Impl DefaultCallbacks on our side

Auto merge of #5155 - krishna-veerareddy:remove-unused-imports, r=flip1995

Remove unnecessary `Lint{Array|Pass}` imports

changelog: none

Remove unnecessary imports from tests

Remove unnecessary imports from documentation

Remove unnecessary imports from `new_lint` tool

Auto merge of #5151 - flip1995:fix_run_pass_tests, r=flip1995

Fix run-pass tests when CARGO_TARGET_DIR is not set

r? @lzutao

I got

```
thread '[ui] ui/crashes/ice-1969.rs' panicked at 'failed to exec `"target/debug/test_build_base/crashes/ice-1969.stage-id"`: Os { code: 2, kind: NotFound, message: "No such file or directory" }', /home/pkrones/.cargo/registry/src/github.com-1ecc6299db9ec823/compiletest_rs-0.4.0/src/runtest.rs:1350:25
```

on tests with `// run-pass` locally. The reason for this is, that I haven't set `CARGO_TARGET_DIR` and then `"target"` was used as the `CARGO_TARGET_DIR`. It seems, that `compiletest-rs` cannot deal with relative paths for `// run-pass` tests.

changelog: none

Use current_dir instead of cargo_metadata

Co-Authored-By: lzutao <taolzu@gmail.com>

Rustup to rust-lang/rust#68725

Fix run-pass tests when CARGO_TARGET_DIR is not set

Auto merge of #5148 - krishna-veerareddy:issue-5147-option-env-unwrap, r=flip1995

Add `option-env-unwrap` lint

changelog: Add `option-env-unwrap` lint

Fixes #5147

Use `PATH` environment variable for testing

The tests were failing on windows because the `HOME` env variable
doesn't exist on it so using the `PATH` variable to test instead.

Auto merge of #5150 - krishna-veerareddy:rustup-69004, r=flip1995

Rustup to rust-lang/rust#68911

changelog: none

Rustup to rust-lang/rust#68911

Allow `option-env-unwrap` within external macros

Account for `expect` being used to unwrap `Option`

Auto merge of #5145 - lzutao:rmeta, r=flip1995

Fix error E0460 when compiled on Rustc repo

Sadly, this mostly reverts  #5121. Now I use HashMap to only store one rlib
per crate. But that would not work when non-compatible version of the same crate show up.

changelog: none

Fix error E0460 when compiled on Rustc repo

Auto merge of #5149 - lzutao:rustup-65232, r=flip1995

Rustup "index ReEmpty by universe"

cc https://github.com/rust-lang/rust/pull/65232

changelog: none

Rustup "index ReEmpty by universe"

Add `option-env-unwrap` lint

Auto merge of #5140 - lzutao:cleanup-replace, r=flip1995

Minor edit to text region replacement

changelog: none

Auto merge of #5146 - lzutao:rustup, r=flip1995

Rustup https://github.com/rust-lang/rust/pull/67359

cc https://github.com/rust-lang/rust/pull/67359

changelog: none

Update CARGO_TARGET_DIR

Rustup https://github.com/rust-lang/rust/pull/67359

dev: Move DOCS_LINK out of lazy_static and reuse it

dev: Make UpdateMode a copy type

dev: Prefer `fs::read*` and improvement to replace text region

Auto merge of #5134 - flip1995:snippet_block, r=phansch

Make it possible to correctly indent snippet_block snippets

This adds a `indent_relative_to` arg to the `{snippet,expr}_block` functions. This makes it possible to keep the correct indentation of block like suggestions.

In addition, this makes the `trim_multiline` function private and adds a `indent_of` function, to get the indentation of the first line of a span.

The suggestion of `needless_continue` cannot be made auto applicable, since it would be also necessary to remove code following the linted expression. (Well, maybe it is possible, but I don't know how to do it. Expanding the suggestion span to the last expression, that should be removed didn't work)

changelog: Improve suggestions, when blocks of code are involved

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

Rustup to rust-lang/rust#68788

cc rust-lang/rust#68901

changelog: none

Rustup to rust-lang/rust#68788

Rename ctx->cx in needless_continue

Improve help message in needless_continue

Fix rebase fallout

Document the indent_relative_to arg of snippet_block

Update remaining test files

Move tests to the end of the file in utils mod.rs

Update needless_continue stderr

Rewrite suggestion generation of needless_continue

Update block_in_if_condition test files

Make block_in_if_condition auto applicable

Optionally indent snippet_block relative to an Expr

Auto merge of #5139 - lzutao:linecount, r=llogiq

dev: Use bytecount for faster line count

changelog: none

Auto merge of #5125 - Areredify:too_many_bools, r=flip1995

Port mitsuhiko's excessive bools lints

Closes #4 .
changelog: add `struct_excessive_bools` and `fn_params_excessive_bools` lints.
I moved is_trait_impl_item check because at first I implemented it as a late pass for some reason but then I realized it's actually an early lint. But it's a useful function to have, should I move it into a separate pr?

dev: Use bytecount for faster line count

Auto merge of #5132 - JohnTitor:fix-fp-in-unwrap-lint, r=flip1995

Do not lint `unnecessary_unwrap` in external macros

Fixes #5131

I think we shouldn't lint `{panicking, unnecessary}_unwrap` in macros, not just `assert!`.

changelog: Fix false positive in `unnecessary_unwrap`

Auto merge of #5121 - lzutao:comptest, r=flip1995

Improve compile_test and dogfood tests

* Hopefully this finally resolves the "multiple matching crates" error, e.g. #4015
* This handle some convenient CARGO env like CARGO_TARGET_DIR and CARGO_BUILD_TARGET

changelog: none

add excessive bools lints

Use lazy_static

move is_trait_impl_item check from functions.rs to utils

Add `serde_derive` to the need-to-be-disambiguated-crates list

Fix dogfood to use cargo mod too

compile-test: Handle CARGO_TARGET_DIR and transitive deps

Auto merge of #5137 - lzutao:race, r=flip1995

Prevent failing to restart setup-toolchain

If `rustup-toolchain-install-master` fails to install master
toolchain (in case `rustup update` executes at the same remove the
tmp directory), `cargo search` (which use master toolchain after
`rustup override` command) will fail. This leads to setup-toolchain
fails too.

changelog: none

Prevent failing to restart setup-toolchain

If `rustup-toolchain-install-master` fails to install master
toolchain (in case `rustup update` executes at the same remove the
tmp directory), `cargo search` (which use master toolchain after
`rustup override` command) will fail. This leads to setup-toolchain
fails too.

Auto merge of #5135 - lzutao:config, r=flip1995

Few improvement to `utils::conf` module

* Fix a few typos
* Handle Option<&Path> early
* Use `env::var_os` when possible

changelog: none

Few improvement to `utils::conf` module

* Fix a few typos
* Handle Option<&Path> early
* Use `env::var_os` when possible

Auto merge of #5061 - ThibsG:UselessMatch3664, r=flip1995

Add new lint: match with a single binding statement

This lint catches `match` statements that binds to only one block and could be rewritten using a simple `let` statement.

Lint name: MATCH_SINGLE_BINDING

fixes: #3664

changelog: add lint for match with single binding statement

Manage macros case + move to MaybeIncorrect when binding values

Merge fixes

Add wild and struct handling

Use span_lint_and_sugg + move infaillible lint

 - moving infaillible lint to prevent collisions