ref(api): Only allow dead code where needed

We added a blanket `allow(dead_code)` for the entire api module in #2096 because of some new clippy lints which were failing.

Here, we remove the blanket `allow(dead_code)` and only allow dead code for the specific lines where it occurs. It might not be possible to remove these because serde might rely on their presence.

src/api/mod.rs

@@ -1,5 +1,3 @@
-#![allow(dead_code)] // a ton of fields might be unused
-
 //! This module implements the API access to the Sentry API as well
 //! as some other APIs we interact with.  In particular it can talk
 //! to the GitHub API to figure out if there are new releases of the
@@ -2025,6 +2023,7 @@ pub struct AuthDetails {
 #[derive(Deserialize, Debug)]
 pub struct User {
     pub email: String,
+    #[allow(dead_code)]
     pub id: String,
 }
 
@@ -2108,6 +2107,7 @@ pub struct UpdatedRelease {
 #[derive(Debug, Deserialize)]
 pub struct ReleaseInfo {
     pub version: String,
+    #[allow(dead_code)]
     pub url: Option<String>,
     #[serde(rename = "dateCreated")]
     pub date_created: DateTime<Utc>,
@@ -2172,6 +2172,7 @@ pub struct DebugInfoData {
     #[serde(default, rename = "type")]
     pub kind: Option<ObjectKind>,
     #[serde(default)]
+    #[allow(dead_code)]
     pub features: Vec<String>,
 }
 
@@ -2188,6 +2189,7 @@ pub struct DebugInfoFile {
     #[serde(rename = "cpuName")]
     pub cpu_name: String,
     #[serde(rename = "sha1")]
+    #[allow(dead_code)]
     pub checksum: String,
     #[serde(default)]
     pub data: DebugInfoData,
@@ -2299,20 +2301,25 @@ pub struct Organization {
     #[serde(rename = "require2FA")]
     pub require_2fa: bool,
     #[serde(rename = "requireEmailVerification")]
+    #[allow(dead_code)]
     pub require_email_verification: bool,
+    #[allow(dead_code)]
     pub features: Vec<String>,
 }
 
 #[derive(Deserialize, Debug)]
 pub struct Team {
+    #[allow(dead_code)]
     pub id: String,
+    #[allow(dead_code)]
     pub slug: String,
     pub name: String,
 }
 
 #[derive(Deserialize, Debug)]
 pub struct ProjectSlugAndName {
     pub slug: String,
+    #[allow(dead_code)]
     pub name: String,
 }
 
@@ -2344,8 +2351,10 @@ pub struct Repo {
     pub name: String,
     pub url: Option<String>,
     pub provider: RepoProvider,
+    #[allow(dead_code)]
     pub status: String,
     #[serde(rename = "dateCreated")]
+    #[allow(dead_code)]
     pub date_created: DateTime<Utc>,
 }
 
@@ -2387,6 +2396,7 @@ pub struct ProcessedEvent {
     #[serde(default)]
     pub title: String,
     #[serde(default, skip_serializing_if = "Option::is_none")]
+    #[allow(dead_code)]
     pub project: Option<u64>,
     #[serde(default, skip_serializing_if = "Option::is_none")]
     pub release: Option<String>,
@@ -2449,6 +2459,7 @@ impl fmt::Display for ProcessedEventTag {
 
 #[derive(Clone, Debug, Deserialize)]
 pub struct Region {
+    #[allow(dead_code)]
     pub name: String,
     pub url: String,
 }