wip

Author: ekump

.github/actions/clippy-annotation-reporter/src/commenter.rs

@@ -89,7 +89,6 @@ async fn find_existing_comment(
                 break;
             }
             Err(e) => {
-                // TODO: EK - Handle error more gracefully
                 error!("Warning: Failed to fetch next page of comments: {}", e);
                 break;
             }

.github/actions/clippy-annotation-reporter/src/config.rs

@@ -157,7 +157,7 @@ impl Config {
         // Use provided values from args if available, otherwise use context
         // TODO: EK - FIX THIS CLONE
         let repository = args.clone().repo.unwrap_or(github_ctx.repository);
-
+        warn!("using args: {:?}", args);
         let pr_number = match args.pr {
             Some(pr) => pr,
             None => {

.github/actions/clippy-annotation-reporter/src/main.rs

@@ -1,10 +1,10 @@
 // Copyright 2024-Present Datadog, Inc. https://www.datadoghq.com/
 // SPDX-License-Identifier: Apache-2.0
 
-use std::env;
 use anyhow::{Context as _, Result};
 use log::info;
 use octocrab::Octocrab;
+use std::env;
 
 mod analyzer;
 mod commenter;
@@ -22,14 +22,13 @@ async fn main() -> Result<()> {
     if env::var("RUST_LOG").is_err() {
         env::set_var("RUST_LOG", &log_level);
     }
-    
+
     env_logger::init();
 
     info!("Clippy Annotation Reporter starting...");
 
     let config = ConfigBuilder::new().build()?;
 
-    // TODO: EK - Should we use context here?
     let octocrab = Octocrab::builder()
         .personal_token(config.token.clone())
         .build()

datadog-trace-protobuf/src/remoteconfig.rs

@@ -3,15 +3,17 @@
 
 use serde::{Deserialize, Serialize};
 // This file is @generated by prost-build.
-#[derive(Deserialize, Serialize, Clone, PartialEq, ::prost::Message)]
+#[derive(Deserialize, Serialize)]
+#[derive(Clone, PartialEq, ::prost::Message)]
 pub struct File {
     #[prost(string, tag = "1")]
     pub path: ::prost::alloc::string::String,
     #[prost(bytes = "vec", tag = "2")]
     #[serde(with = "serde_bytes")]
     pub raw: ::prost::alloc::vec::Vec<u8>,
 }
-#[derive(Deserialize, Serialize, Clone, PartialEq, ::prost::Message)]
+#[derive(Deserialize, Serialize)]
+#[derive(Clone, PartialEq, ::prost::Message)]
 pub struct Client {
     #[prost(message, optional, tag = "1")]
     pub state: ::core::option::Option<ClientState>,
@@ -33,7 +35,8 @@ pub struct Client {
     #[prost(bytes = "vec", tag = "11")]
     pub capabilities: ::prost::alloc::vec::Vec<u8>,
 }
-#[derive(Deserialize, Serialize, Clone, PartialEq, ::prost::Message)]
+#[derive(Deserialize, Serialize)]
+#[derive(Clone, PartialEq, ::prost::Message)]
 pub struct ClientTracer {
     #[prost(string, tag = "1")]
     pub runtime_id: ::prost::alloc::string::String,
@@ -52,7 +55,8 @@ pub struct ClientTracer {
     #[prost(string, repeated, tag = "7")]
     pub tags: ::prost::alloc::vec::Vec<::prost::alloc::string::String>,
 }
-#[derive(Deserialize, Serialize, Clone, PartialEq, ::prost::Message)]
+#[derive(Deserialize, Serialize)]
+#[derive(Clone, PartialEq, ::prost::Message)]
 pub struct ClientAgent {
     #[prost(string, tag = "1")]
     pub name: ::prost::alloc::string::String,
@@ -65,7 +69,8 @@ pub struct ClientAgent {
     #[prost(string, repeated, tag = "5")]
     pub cws_workloads: ::prost::alloc::vec::Vec<::prost::alloc::string::String>,
 }
-#[derive(Deserialize, Serialize, Clone, PartialEq, ::prost::Message)]
+#[derive(Deserialize, Serialize)]
+#[derive(Clone, PartialEq, ::prost::Message)]
 pub struct ConfigState {
     #[prost(string, tag = "1")]
     pub id: ::prost::alloc::string::String,
@@ -78,7 +83,8 @@ pub struct ConfigState {
     #[prost(string, tag = "5")]
     pub apply_error: ::prost::alloc::string::String,
 }
-#[derive(Deserialize, Serialize, Clone, PartialEq, ::prost::Message)]
+#[derive(Deserialize, Serialize)]
+#[derive(Clone, PartialEq, ::prost::Message)]
 pub struct ClientState {
     #[prost(uint64, tag = "1")]
     pub root_version: u64,
@@ -93,14 +99,16 @@ pub struct ClientState {
     #[prost(bytes = "vec", tag = "6")]
     pub backend_client_state: ::prost::alloc::vec::Vec<u8>,
 }
-#[derive(Deserialize, Serialize, Clone, PartialEq, ::prost::Message)]
+#[derive(Deserialize, Serialize)]
+#[derive(Clone, PartialEq, ::prost::Message)]
 pub struct TargetFileHash {
     #[prost(string, tag = "1")]
     pub algorithm: ::prost::alloc::string::String,
     #[prost(string, tag = "3")]
     pub hash: ::prost::alloc::string::String,
 }
-#[derive(Deserialize, Serialize, Clone, PartialEq, ::prost::Message)]
+#[derive(Deserialize, Serialize)]
+#[derive(Clone, PartialEq, ::prost::Message)]
 pub struct TargetFileMeta {
     #[prost(string, tag = "1")]
     pub path: ::prost::alloc::string::String,
@@ -109,14 +117,16 @@ pub struct TargetFileMeta {
     #[prost(message, repeated, tag = "3")]
     pub hashes: ::prost::alloc::vec::Vec<TargetFileHash>,
 }
-#[derive(Deserialize, Serialize, Clone, PartialEq, ::prost::Message)]
+#[derive(Deserialize, Serialize)]
+#[derive(Clone, PartialEq, ::prost::Message)]
 pub struct ClientGetConfigsRequest {
     #[prost(message, optional, tag = "1")]
     pub client: ::core::option::Option<Client>,
     #[prost(message, repeated, tag = "2")]
     pub cached_target_files: ::prost::alloc::vec::Vec<TargetFileMeta>,
 }
-#[derive(Deserialize, Serialize, Clone, PartialEq, ::prost::Message)]
+#[derive(Deserialize, Serialize)]
+#[derive(Clone, PartialEq, ::prost::Message)]
 pub struct ClientGetConfigsResponse {
     #[prost(bytes = "vec", repeated, tag = "1")]
     #[serde(with = "crate::serde")]