base-tests: don't print all commands to stdout during the loop

Auto merge of #3533 - phansch:add_to_doc_valid_idents, r=phansch

Add 'CamelCase' to doc_valid_idents

Fixes #2395

Auto merge of #3529 - matthiaskrgr:rustfmt_tests, r=phansch

rustfmt tests

also fails CI if tests turn out to not be formatted properly.

test reformatting: revert more questionable changes done by rustfmt and add #[rustfmt::skip]

Add 'CamelCase' to doc_valid_idents

Auto merge of #3532 - maxencefrenette:fix-contributing-md, r=oli-obk

Remove dead link in CONTRIBUTING.md

I don't think there was an issue for this, but this is just removing a dead link in CONTRIBUTING.md. The _Linting Clippy with your local changes_ section seems to have been removed without updating the table of contents.

I'm not sure if I should have opened an issue, but it seemed like a trivial fix to me.

Remove dead link in CONTRIBUTING.md

Auto merge of #3527 - phansch:update_readme2, r=matthiaskrgr

readme: tool lints are stable now

None

Merge branch 'master' into rustfmt_tests

tests: fix more cases where rustfmt would have hurt the tests

tests: revert some changs and add further rustfmt::skip attributes.

s/clippy/Clippy in readme

Also add note about using -W clippy::lint_group

readme: tool lints are stable now

Auto merge of #3511 - phansch:remove_allow_doc_markdown, r=phansch

Remove allow(doc_markdown) in excessive_precision.rs

Closes #3299

Auto merge of #3518 - sinkuu:redundant_clone_tw, r=phansch

Lint redundant clone of fields

Makes `redundant_clone` warn on unnecessary `foo.field.clone()` sometimes (it can detect an unnecessary clone only if the base of projection, `foo` in this case, is not used at all after that). This is enough for cases like `returns_tuple().0.clone()`.

test formatting: don't format tests/ui/formatting.rs

update test stderr

Auto merge of #3524 - phansch:update_contributing, r=phansch

Small updates to CONTRIBUTING.md

None

Document bors/homu

Small updates to CONTRIBUTING.md

Only check the assignment found at last

If there are more than one such assignment, the last one may be
the one supplied to `clone` method.
Makes `find_stmt_assigns_to` internally reverses the iterator to make
the intent to "iterate statements backward" clear.

Add comment and rename

Auto merge of #3522 - matthiaskrgr:target_dir, r=phansch

travis: base-tests: share CARGO_TARGET_DIR between check runs of subcrates to avoid unnecessarily recompiling deps.

Apply redundant_clone on clippy

Lint redundant clone of projection

rustfmt tests

travis: base-tests: share CARGO_TARGET_DIR between check runs of subcrates to avoid unneccessarily recompiling deps.

base tests: assert that tests are properly formatted.

Auto merge of #3520 - matthiaskrgr:test_rustfmt_skip, r=phansch

add rustfmt::skip attributes to a couple of tests

part of rustfmt'ing test code

update line numbers of tests

add rustfmt::skip attributes to some tests

Merge pull request #3517 from phansch/add_fast_finish

Add `fast_finish` to travis matrix

Merge pull request #3516 from phansch/remove_run_rustfix_annotation

Remove a run-rustfix annotation (for now)

Fix test

`if true` is recognized by MIR optimization.

Add `fast_finish` to travis matrix

This means we don't have to wait for `allowed_failures` builds to
complete. It should save us ~10 minutes until we remove the windows
build from `allowed_failures`.

Remove a run-rustfix annotation (for now)

Starting to work on #2376, this annotation got in the way. Going to
remove it for now.

Merge pull request #3515 from matthiaskrgr/fmt

run rustfmt

run rustfmt

Cleanup

Merge pull request #3512 from matthiaskrgr/rustup

rustup https://github.com/rust-lang/rust/pull/56502 ( .hir -> .hir() )

s/rustfmt-preview/rustfmt/

Swap if branches

Fix c_void false positive caused by libc refactoring

The path of `libc::c_void` has changes in https://github.com/rust-lang/libc/commit/5c1a6b8a6d558882927a0816d91c01b9c2a88018
The DefId path is now always platform specific like
`libc::windows::c_void`. This fixes our c_void detection to only check
the first and last elements.

rustup https://github.com/rust-lang/rust/pull/56502 ( .hir -> .hir() )

Merge pull request #3336 from HMPerson1/clone_on_copy_deref

Fix `clone_on_copy` not detecting derefs sometimes

Remove allow(doc_markdown) in excessive_precision.rs

Merge pull request #3507 from flying-sheep/patch-1

typo: emum → enum

typo: emum → enum

Merge pull request #3257 from o01eg/remove-sysroot

Don't try to determine sysroot. rustc_driver will use default value.

Merge pull request #3501 from matthiaskrgr/rustup

rustup clippy build with latest rustc

Merge pull request #3502 from rust-lang/nrc-patch-1

Remove -preview suffix from README

Remove -preview suffix from README

rustup clippy build with latest rustc
(breakage due to https://github.com/rust-lang/rust/commit/08f8faedd0e30f45762afbb8d4873f7041e7462c )

Fixes #3500

Merge pull request #3497 from daxpedda/master

Fix bug in `implicit_return`.

Forgot to remove some debugging code ...

Improved code noted by clippy.

Merge pull request #3496 from fkohlgrueber/macro-in-closure-in-if-cond

fix #3482 and add ui test for it

Fix bug in `implicit_return`.
Bug was already covered by test, but test was not checked for.

Fix format.

Add sysroot gettinh code to dogfood tests.

fix #3482 and add ui test for it

Add sysroot getting code to tests.

Don't try to determine sysroot. rustc_driver will use default value.

Merge pull request #3494 from daxpedda/master

Added `IMPLICIT_RETURN` lint.

Merge pull request #3444 from waynr/fix-dogfood-tests

Fix dogfood tests

Don't change current working directory of cargo tests

Use cargo's "PROFILE" envvar and set CLIPPY_DOGFOOD

Use dogfood_runner for deterministic test ordering

Remove unnecessary documentation

Fix dogfood tests.

Added additional reasoning to `Why is this bad?`.
Added comment to explain usage of MIR.

Renamed to `implicit_return`.
Covered all other kinds besides `ExprKind::Lit`.
Added check for replacing `break` with `return`.

Merge pull request #3495 from flip1995/tykind_fix

Fix usage of ty::TyKind

Fix ty::TyKind usage

Fix TyKind::<kind> usage in codebase

Appeasing the Test Gods.
Seems I'm not smart enough to run the tests locally before committing.

Renamed `forced_return` to `missing_returns`.
Better clarification in the docs.
Ran `update_lints`.

Added `FORCED_RETURN` lint.

Merge pull request #3490 from phansch/extract_single_match_else_ui_test

Extract single_match_else UI test

Merge pull request #3473 from lucasloisp/additional-bool-comparisons

Adds inequality cases to bool comparison (#3438)

Merge pull request #3488 from phansch/fix_cargo_lint_bug

Fix wildcard_dependencies false positive

Extract single_match_else UI test

There's only one test currently.
I also updated the lint doc with a 'good' example and changed the lint
help text a bit.

cc #2038

cargo fmt

Adds inequality cases to bool comparison lint

The lint now checks cases like `y != true`

Merge pull request #3478 from dtolnay/setlen

Remove unsafe_vector_initialization lint

Remove unsafe_vector_initialization lint

Merge pull request #3479 from dtolnay/literal

Downgrade large_digit_groups to pedantic

Fix wildcard_dependencies false positive

This now only checks for wildcard_dependencies if the source is a
non-git source.

I tried adding a compiletest suite for the cargo lints, but I was unable
to override the `Cargo.toml` of the original executable.

I tested this manually by modifying the main `Cargo.toml`.

Fixes #3458

Merge pull request #3487 from phansch/fix_fmt

Fix rustfmt format

Fix rustfmt format

Merge pull request #3486 from phansch/make_windows_travis_run_more_stuff

Travis: Add rustc sysroot bin to PATH for windows build

Disable incremental build for windows

Testing if this speeds up compilation time for the Windows CI build.

Travis: Add rustc sysroot bin to PATH for windows build

Merge pull request #3483 from phansch/mention_triage_procedure_in_contributing

Mention triage procedure in contributing.md

Merge pull request #3485 from phansch/fix_some_nursery_links

Fix some nursery links

Fix some nursery links

Merge pull request #3484 from phansch/fix_windows_lineendings

git: Enforce LF lineendings for everything

Enforce LF lineendings for everything

Someone on discord reported issues with UI tests.

This should make sure that git never automatically converts lineendings
for text files to `CRLF`. They should always be `LF` now.

Probably this means that we can stop using dos2unix for #3306, too.

Taken from [Rust's .gitattributes file](https://github.com/rust-lang/rust/blob/master/.gitattributes).

Mention triage procedure in contributing.md