feat(uploader): add platform metadata

CodSpeedHQ · Nov 28, 2023 · e44da76 · e44da76
1 parent cfa5f21
commit e44da76
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/src/ci_provider/github_actions/provider.rs b/src/ci_provider/github_actions/provider.rs
@@ -139,10 +139,9 @@ impl CIProvider for GitHubActionsProvider {
             ref_: self.ref_.clone(),
 
             // TODO: refactor in a default implementation of the trait, as it will be the same for all providers
+            platform: self.get_provider_slug().into(),
             runner: Runner {
                 name: "codspeed-runner".into(),
-                // TODO add back when integrating another provider
-                // platform: self.get_provider_slug().into(),
                 version: VERSION.to_string(),
             },
             tokenless: config.token.is_none(),

diff --git a/...ner__ci_provider__github_actions__provider__tests__fork_pull_request_upload_metadata.snap b/...ner__ci_provider__github_actions__provider__tests__fork_pull_request_upload_metadata.snap
@@ -24,5 +24,6 @@ expression: upload_metadata
   "runner": {
     "name": "codspeed-runner",
     "version": "[version]"
-  }
+  },
+  "platform": "github-actions"
 }
diff --git a/...d_runner__ci_provider__github_actions__provider__tests__pull_request_upload_metadata.snap b/...d_runner__ci_provider__github_actions__provider__tests__pull_request_upload_metadata.snap
@@ -24,5 +24,6 @@ expression: upload_metadata
   "runner": {
     "name": "codspeed-runner",
     "version": "[version]"
-  }
+  },
+  "platform": "github-actions"
 }
diff --git a/src/uploader/interfaces.rs b/src/uploader/interfaces.rs
@@ -25,6 +25,7 @@ pub struct UploadMetadata {
     pub profile_md5: String,
     pub gh_data: Option<GhData>,
     pub runner: Runner,
+    pub platform: String,
 }
 
 #[derive(Deserialize, Serialize, Debug, Clone)]
@@ -46,8 +47,6 @@ pub struct Sender {
 #[serde(rename_all = "camelCase")]
 pub struct Runner {
     pub name: String,
-    // TODO add back when integrating another provider
-    // pub platform: String,
     pub version: String,
 }