]> git.lizzy.rs Git - rust.git/commit
Auto merge of #7187 - camsteffen:avoid-break-exported, r=flip1995,phansch
authorbors <bors@rust-lang.org>
Thu, 27 May 2021 04:49:56 +0000 (04:49 +0000)
committerbors <bors@rust-lang.org>
Thu, 27 May 2021 04:49:56 +0000 (04:49 +0000)
commit9c4651f2eeedc6f17f970d05699706c527c7a658
tree9809d36b0ebcdfa4c843ad41a0a6a4c61fa34467
parentf205dd176a277457d3a5052565807b31e9391df4
parent6eea598645be5489f518f91e4b80a0f04a315fda
Auto merge of #7187 - camsteffen:avoid-break-exported, r=flip1995,phansch

Add avoid_breaking_exported_api config option

changelog: Add `avoid_breaking_exported_api` config option for [`enum_variant_names`], [`large_types_passed_by_value`], [`trivially_copy_pass_by_ref`], [`unnecessary_wraps`], [`upper_case_acronyms`] and [`wrong_self_convention`].

changelog: Deprecates [`pub_enum_variant_names`] and [`wrong_pub_self_convention`] as the non-pub variants are now configurable.

changelog: Fix various false negatives for `pub` items that are not exported from the crate.

A couple changes to late passes in order to use `cx.access_levels.is_exported` rather than `item.vis.kind.is_pub`.

I'm not sure how to better document the config option or lints that are (not) affected (see comments in #6806). Suggestions are welcome. cc `@rust-lang/clippy`

I added `/clippy.toml` to use the config internally and `/tests/clippy.toml` to maintain a default config in ui tests.

Closes #6806
Closes #4504
clippy_lints/src/lib.rs