Refactor createJobSummary function to include total usage statistics …

…and improve code readability
austenstone · Apr 23, 2024 · e9ce023 · e9ce023
1 parent c90ec10
commit e9ce023
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/dist/index.js b/dist/index.js
diff --git a/src/index.ts b/src/index.ts
@@ -137,10 +137,10 @@ const createJobSummary = async (data: CopilotUsageResponse) => {
 
   await summary
     .addHeading(`Copilot Usage Results for ${data[0].day} to ${data[data.length - 1].day}`)
-    .addHeading(`Suggestions: ${totalSuggestionsCount}`)
-    .addHeading(`Acceptances: ${totalAcceptanceCount}`)
+    .addHeading(`Suggestions: ${totalSuggestionsCount.toLocaleString()}`)
+    .addHeading(`Acceptances: ${totalAcceptanceCount.toLocaleString()}`)
     .addHeading(`Acceptance Rate: ${totalAcceptanceRate}%`)
-    .addHeading(`Lines of Code Accepted: ${totalLinesOfCodeAccepted}`)
+    .addHeading(`Lines of Code Accepted: ${totalLinesOfCodeAccepted.toLocaleString()}`)
     .addRaw(getXyChartAcceptanceRate(data))
     .addRaw(getXyChartDailyActiveUsers(data))
     .addHeading('Language Usage')