Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

submodules: update clippy from b1d03437 to 3971c424 #55484

Closed
wants to merge 1 commit into from

Conversation

matthiaskrgr
Copy link
Member

Should make clippy tests green again on rust-toolstate

Changes:

compiletest: clean rmeta data (from "cargo check") before running compiletest.
dependencies: bump compiletest-rs from git to 0.3.16
UI test cleanup: Extract explicit_counter_loop tests
UI test cleanup: Extract unnecessary_operation tests
appveyor: use rustc nightly instead of master
pin compiletest dependency to git version (12c980f47971b5ba6beb7cb2ffebf8b32f6766ea) while we are waiting for a new release
rustup: fix build with rustc 1.31.0-nightly (cae6efc37 2018-10-27)
Disable rust master toolchain build temporarily
Don't expand macro in or_fun_call suggestion
Don't expand macro in single_match suggestion
Don't expand macro in identity_conversion suggestion
slightly simplify integration tests
simplify ci base-tests
fix: correctly reconstruct raw strings
fix: extra semicolon, only create callsite once
Fix string_lit_as_bytes lint for macros
travis: work around temporary test failure due to rustc crashing on hyper.
ci: allow all branches except trying.tmp and staging.tmp to be built
Move in_macro check
Use BasicBlockData::terminator
Refactor
Use lint_root
Implement visit_basic_block_data
update_references indexing_slicing
Run update_lints
Add redundant_clone lint
Revert "new_ret_no_self: add sample from #3313 to Known Problems section."
Add tests for more than one level of reference
Add test case for `mem::discriminant` inside a macro
Add `Applicability`
Add lint for calling `mem::discriminant` on a non-enum type
Check existential types in `use_self`
Disable arithmetic lints in constant items
Fix warnings introduced by #3349
new_ret_no_self added test cases
Replace remaining `krate.span` with `DUMMY_SP`
Use DUMMY_SP in multiple_crate_versions
Some fixes for wildcard_dependencies
Run util/update_lints.py
Minor changes on clippy_lints/src/wildcard_dependencies.rs
Lint for wildcard dependencies in Cargo.toml
Don't emit `new_without_default_derive` if an impl of Default exists
Fix inspector pass documentation
Add branch configuration to appveyor.yml
Setup bors
new_ret_no_self added test cases
new_ret_no_self walk return type to check for self
Update `ui/for_loop` test output
Check for known array length in `needless_range_loop`
new_ret_no_self correct false positive on raw pointer return types
new_ret_no_self correct linting of tuple return types
out_of_bounds_indexing improved reporting of out of bounds value
out_of_bounds_indexing refactoring
OUT_OF_BOUNDS_INDEXING fix #3102 false negative

@rust-highfive
Copy link
Collaborator

r? @aturon

(rust_highfive has picked a reviewer for you, use r? to override)

@rust-highfive
Copy link
Collaborator

⚠️ Warning ⚠️

  • These commits modify submodules.

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 29, 2018
Should make clippy tests green again on rust-toolstate

Changes:
````
compiletest: clean rmeta data (from "cargo check") before running compiletest.
dependencies: bump compiletest-rs from git to 0.3.16
UI test cleanup: Extract explicit_counter_loop tests
UI test cleanup: Extract unnecessary_operation tests
appveyor: use rustc nightly instead of master
pin compiletest dependency to git version (12c980f47971b5ba6beb7cb2ffebf8b32f6766ea) while we are waiting for a new release
rustup: fix build with rustc 1.31.0-nightly (cae6efc 2018-10-27)
Disable rust master toolchain build temporarily
Don't expand macro in or_fun_call suggestion
Don't expand macro in single_match suggestion
Don't expand macro in identity_conversion suggestion
slightly simplify integration tests
simplify ci base-tests
fix: correctly reconstruct raw strings
fix: extra semicolon, only create callsite once
Fix string_lit_as_bytes lint for macros
travis: work around temporary test failure due to rustc crashing on hyper.
ci: allow all branches except trying.tmp and staging.tmp to be built
Move in_macro check
Use BasicBlockData::terminator
Refactor
Use lint_root
Implement visit_basic_block_data
update_references indexing_slicing
Run update_lints
Add redundant_clone lint
Revert "new_ret_no_self: add sample from rust-lang#3313 to Known Problems section."
Add tests for more than one level of reference
Add test case for `mem::discriminant` inside a macro
Add `Applicability`
Add lint for calling `mem::discriminant` on a non-enum type
Check existential types in `use_self`
Disable arithmetic lints in constant items
Fix warnings introduced by rust-lang#3349
new_ret_no_self added test cases
Replace remaining `krate.span` with `DUMMY_SP`
Use DUMMY_SP in multiple_crate_versions
Some fixes for wildcard_dependencies
Run util/update_lints.py
Minor changes on clippy_lints/src/wildcard_dependencies.rs
Lint for wildcard dependencies in Cargo.toml
Don't emit `new_without_default_derive` if an impl of Default exists
Fix inspector pass documentation
Add branch configuration to appveyor.yml
Setup bors
new_ret_no_self added test cases
new_ret_no_self walk return type to check for self
Update `ui/for_loop` test output
Check for known array length in `needless_range_loop`
new_ret_no_self correct false positive on raw pointer return types
new_ret_no_self correct linting of tuple return types
out_of_bounds_indexing improved reporting of out of bounds value
out_of_bounds_indexing refactoring
OUT_OF_BOUNDS_INDEXING fix rust-lang#3102 false negative
````
@matthiaskrgr matthiaskrgr changed the title submodules: update cliopy from b1d03437 to 3971c424 submodules: update clippy from b1d03437 to 3971c424 Oct 29, 2018
@kennytm
Copy link
Member

kennytm commented Oct 30, 2018

🤔 There is also #55496 and already r+'ed

@matthiaskrgr
Copy link
Member Author

Oh well...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants