allow needless raw string hashes and unnecessary wraps (#1969)

This change updates our root clippy config and removes the individual
overrides for `needless_raw_string_hashes`. It also preemptively allows
`unnecessary_wraps` in anticipation of #1954

Author: sezna

Cargo.toml

@@ -88,6 +88,8 @@ unwrap_used = "warn"
 missing_errors_doc = "allow"
 missing_panics_doc = "allow"
 module_name_repetitions = "allow"
+needless_raw_string_hashes = "allow"
+unnecessary_wraps = "allow"
 
 [profile.release]
 debug = "limited"

compiler/qsc/src/interpret/debug/tests.rs

@@ -1,8 +1,6 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
-#![allow(clippy::needless_raw_string_hashes)]
-
 use indoc::indoc;
 use miette::Result;
 use qsc_data_structures::{language_features::LanguageFeatures, target::TargetCapabilityFlags};

compiler/qsc/src/interpret/debugger_tests.rs

@@ -1,8 +1,6 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
-#![allow(clippy::needless_raw_string_hashes)]
-
 use crate::interpret::Debugger;
 use crate::line_column::Encoding;
 use qsc_data_structures::language_features::LanguageFeatures;

compiler/qsc/src/interpret/package_tests.rs

@@ -1,8 +1,6 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
-#![allow(clippy::needless_raw_string_hashes)]
-
 use crate::{interpret::Interpreter, packages::BuildableProgram};
 use indoc::indoc;
 use qsc_data_structures::{language_features::LanguageFeatures, target::TargetCapabilityFlags};

compiler/qsc/src/interpret/tests.rs

@@ -1,8 +1,6 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
-#![allow(clippy::needless_raw_string_hashes)]
-
 mod given_interpreter {
     use crate::interpret::{InterpretResult, Interpreter};
     use expect_test::Expect;

compiler/qsc_codegen/src/qsharp/spec_decls.rs

@@ -2,7 +2,6 @@
 // Licensed under the MIT License.
 
 #![allow(clippy::too_many_lines)]
-#![allow(clippy::needless_raw_string_hashes)]
 
 use expect_test::expect;
 use indoc::indoc;

compiler/qsc_codegen/src/qsharp/test_utils.rs

@@ -2,7 +2,6 @@
 // Licensed under the MIT License.
 
 #![allow(clippy::too_many_lines)]
-#![allow(clippy::needless_raw_string_hashes)]
 
 use std::sync::Arc;
 

compiler/qsc_codegen/src/qsharp/tests.rs

@@ -2,7 +2,6 @@
 // Licensed under the MIT License.
 
 #![allow(clippy::too_many_lines)]
-#![allow(clippy::needless_raw_string_hashes)]
 
 use expect_test::expect;
 use indoc::indoc;

compiler/qsc_data_structures/src/namespaces/tests.rs

@@ -4,8 +4,6 @@
 // allowing needless raw hashes here because we auto-update these expected outputs
 // and don't want to risk weird breakages
 
-#![allow(clippy::needless_raw_string_hashes)]
-
 use expect_test::expect;
 
 use super::*;

compiler/qsc_doc_gen/src/generate_docs/tests.rs

@@ -1,8 +1,6 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
-#![allow(clippy::needless_raw_string_hashes)]
-
 use super::generate_docs;
 use expect_test::expect;