Fix build issues (#1625)

* Fix build issues, inherit workspace lints * More fixes * Add empty workspace.lints to resolve services/ errors * services/autorest/Cargo.toml is a workspace
Azure · Mar 13, 2024 · 22ab112 · 22ab112
1 parent e88ec52
commit 22ab112
Show file tree

Hide file tree

Showing 299 changed files with 1,476 additions and 291 deletions.
diff --git a/Cargo.toml b/Cargo.toml
@@ -4,3 +4,5 @@ members = [
   "./eng/test/mock_transport"
 ]
 resolver = "2"
+
+[workspace.lints]
diff --git a/sdk/core/src/error/macros.rs b/sdk/core/src/error/macros.rs
@@ -66,6 +66,7 @@ macro_rules! ensure_ne {
 mod tests {
     use super::super::*;
 
+    #[allow(dead_code)]
     #[derive(Debug, PartialEq, Copy, Clone)]
     struct OperationError;
 

diff --git a/sdk/storage_blobs/examples/list_containers2.rs b/sdk/storage_blobs/examples/list_containers2.rs
@@ -5,6 +5,7 @@ use serde::Serialize;
 
 #[derive(Debug, Clone, Serialize)]
 #[serde(rename_all = "snake_case")]
+#[allow(dead_code)]
 struct SampleEntity {
     pub something: String,
 }

diff --git a/services/Cargo.toml b/services/Cargo.toml
@@ -291,4 +291,7 @@ members = [
   "svc/webpubsub",
 ]
 
-resolver = "2"
+resolver = "2"
+
+[workspace.lints.clippy]
+empty-docs = "allow"
diff --git a/services/autorust/Cargo.toml b/services/autorust/Cargo.toml
@@ -6,3 +6,6 @@ members = [
 ]
 
 resolver = "2"
+
+[workspace.lints]
+
diff --git a/services/autorust/azure-autorust/Cargo.toml b/services/autorust/azure-autorust/Cargo.toml
@@ -8,4 +8,8 @@ publish = false
 [dependencies]
 autorust_codegen = { path = "../codegen" }
 clap = { version = "4.0", features = ["derive"] }
-rayon = "1.8"
+rayon = "1.8"
+
+[lints]
+workspace = true
+
diff --git a/services/autorust/codegen/Cargo.toml b/services/autorust/codegen/Cargo.toml
@@ -31,3 +31,7 @@ cargo_toml = "0.19"
 [dev-dependencies]
 thiserror = "1.0"
 crates_io_api = "0.9"
+
+[lints]
+workspace = true
+
diff --git a/services/autorust/codegen/templates/Cargo.toml.jinja b/services/autorust/codegen/templates/Cargo.toml.jinja
@@ -42,3 +42,6 @@ default_tag = ["{{default_tag}}"]
 {%- for feature in features %}
 "{{feature}}" = []
 {%- endfor %}
+
+[lints]
+workspace = true
diff --git a/services/autorust/openapi/Cargo.toml b/services/autorust/openapi/Cargo.toml
@@ -21,3 +21,7 @@ thiserror = "1.0"
 assert-json-diff = "2.0"
 serde_yaml = "0.9"
 camino = "1.1"
+
+[lints]
+workspace = true
+
diff --git a/services/mgmt/activedirectory/Cargo.toml b/services/mgmt/activedirectory/Cargo.toml
diff --git a/services/mgmt/addons/Cargo.toml b/services/mgmt/addons/Cargo.toml
diff --git a/services/mgmt/adhybridhealthservice/Cargo.toml b/services/mgmt/adhybridhealthservice/Cargo.toml
diff --git a/services/mgmt/adp/Cargo.toml b/services/mgmt/adp/Cargo.toml
diff --git a/services/mgmt/advisor/Cargo.toml b/services/mgmt/advisor/Cargo.toml
diff --git a/services/mgmt/agrifood/Cargo.toml b/services/mgmt/agrifood/Cargo.toml
diff --git a/services/mgmt/alertsmanagement/Cargo.toml b/services/mgmt/alertsmanagement/Cargo.toml
diff --git a/services/mgmt/analysisservices/Cargo.toml b/services/mgmt/analysisservices/Cargo.toml
diff --git a/services/mgmt/apicenter/Cargo.toml b/services/mgmt/apicenter/Cargo.toml
diff --git a/services/mgmt/apimanagement/Cargo.toml b/services/mgmt/apimanagement/Cargo.toml
diff --git a/services/mgmt/app/Cargo.toml b/services/mgmt/app/Cargo.toml
diff --git a/services/mgmt/appcomplianceautomation/Cargo.toml b/services/mgmt/appcomplianceautomation/Cargo.toml
diff --git a/services/mgmt/appconfiguration/Cargo.toml b/services/mgmt/appconfiguration/Cargo.toml
diff --git a/services/mgmt/applicationinsights/Cargo.toml b/services/mgmt/applicationinsights/Cargo.toml
diff --git a/services/mgmt/appplatform/Cargo.toml b/services/mgmt/appplatform/Cargo.toml
diff --git a/services/mgmt/arcdata/Cargo.toml b/services/mgmt/arcdata/Cargo.toml
diff --git a/services/mgmt/attestation/Cargo.toml b/services/mgmt/attestation/Cargo.toml
diff --git a/services/mgmt/authorization/Cargo.toml b/services/mgmt/authorization/Cargo.toml
diff --git a/services/mgmt/automanage/Cargo.toml b/services/mgmt/automanage/Cargo.toml
diff --git a/services/mgmt/automation/Cargo.toml b/services/mgmt/automation/Cargo.toml
diff --git a/services/mgmt/baremetalinfrastructure/Cargo.toml b/services/mgmt/baremetalinfrastructure/Cargo.toml
diff --git a/services/mgmt/batch/Cargo.toml b/services/mgmt/batch/Cargo.toml
diff --git a/services/mgmt/billing/Cargo.toml b/services/mgmt/billing/Cargo.toml
diff --git a/services/mgmt/billingbenefits/Cargo.toml b/services/mgmt/billingbenefits/Cargo.toml
diff --git a/services/mgmt/blockchain/Cargo.toml b/services/mgmt/blockchain/Cargo.toml
diff --git a/services/mgmt/blueprint/Cargo.toml b/services/mgmt/blueprint/Cargo.toml
diff --git a/services/mgmt/botservice/Cargo.toml b/services/mgmt/botservice/Cargo.toml
diff --git a/services/mgmt/cdn/Cargo.toml b/services/mgmt/cdn/Cargo.toml
diff --git a/services/mgmt/changeanalysis/Cargo.toml b/services/mgmt/changeanalysis/Cargo.toml
Original file line number	Diff line number	Diff line change
Expand Up		@@ -6,3 +6,6 @@ members = [
		]

		resolver = "2"

		[workspace.lints]