Use #[expect(clippy::...)] throughout and drop unused supressions (#17537)
Co-authored-by: Claude <noreply@anthropic.com>
This commit is contained in:
@@ -1023,7 +1023,7 @@ impl ManagedPythonDownloadList {
|
||||
// this by parsing into a Map<String, IgnoredAny> which allows any valid JSON on the
|
||||
// value side. (Because it's zero-sized, Clippy suggests Set<String>, but that won't
|
||||
// have the same parsing effect.)
|
||||
#[allow(clippy::zero_sized_map_values)]
|
||||
#[expect(clippy::zero_sized_map_values)]
|
||||
|e| {
|
||||
let source = match json_source {
|
||||
Source::BuiltIn => "EMBEDDED IN THE BINARY".to_owned(),
|
||||
|
||||
@@ -40,7 +40,7 @@ use crate::{
|
||||
use windows::Win32::Foundation::{APPMODEL_ERROR_NO_PACKAGE, ERROR_CANT_ACCESS_FILE, WIN32_ERROR};
|
||||
|
||||
/// A Python executable and its associated platform markers.
|
||||
#[allow(clippy::struct_excessive_bools)]
|
||||
#[expect(clippy::struct_excessive_bools)]
|
||||
#[derive(Debug, Clone)]
|
||||
pub struct Interpreter {
|
||||
platform: Platform,
|
||||
@@ -920,7 +920,7 @@ pub enum InterpreterInfoError {
|
||||
EmscriptenNotPyodide,
|
||||
}
|
||||
|
||||
#[allow(clippy::struct_excessive_bools)]
|
||||
#[expect(clippy::struct_excessive_bools)]
|
||||
#[derive(Debug, Deserialize, Serialize, Clone)]
|
||||
struct InterpreterInfo {
|
||||
platform: Platform,
|
||||
|
||||
@@ -937,7 +937,7 @@ pub fn create_link_to_executable(link: &Path, executable: &Path) -> Result<(), E
|
||||
|
||||
// OK to use `std::fs` here, `fs_err` does not support `File::create_new` and we attach
|
||||
// error context anyway
|
||||
#[allow(clippy::disallowed_types)]
|
||||
#[expect(clippy::disallowed_types)]
|
||||
{
|
||||
std::fs::File::create_new(link)
|
||||
.and_then(|mut file| file.write_all(launcher.as_ref()))
|
||||
|
||||
Reference in New Issue
Block a user