Merge pull request #262 from hossain-khan/renovate/org.jmailen.kotlin…

…ter-3.x

Update plugin org.jmailen.kotlinter to v3.16.0

build.gradle.kts

@@ -4,7 +4,7 @@ plugins {
     // For build.gradle.kts (Kotlin DSL)
     // https://kotlinlang.org/docs/releases.html#release-details
     kotlin("jvm") version "1.9.22"
-    id("org.jmailen.kotlinter") version "3.12.0"
+    id("org.jmailen.kotlinter") version "3.16.0"
     // https://kotlinlang.org/docs/ksp-quickstart.html#use-your-own-processor-in-a-project
     // id("com.google.devtools.ksp") version "1.9.20-1.0.6" // Not needed yet.
     application

src/main/kotlin/PrStatsApplication.kt

@@ -26,7 +26,7 @@ class PrStatsApplication : KoinComponent {
                 pullRequestStatsRepo.stats(
                     repoOwner = repoOwner,
                     repoId = repoId,
-                    prNumber = prNumber
+                    prNumber = prNumber,
                 )
             } catch (e: Exception) {
                 val error = errorProcessor.getDetailedError(e)

src/main/kotlin/StatsGeneratorApplication.kt

@@ -30,7 +30,7 @@ class StatsGeneratorApplication(
      * Get all the available stats formatters - such as ASCII table, CSV writer and so on.
      * @see StatsFormatter
      */
-    private val formatters: List<StatsFormatter>
+    private val formatters: List<StatsFormatter>,
 ) : KoinComponent {
 
     /**

src/main/kotlin/dev/hossain/githubstats/PrAuthorStatsService.kt

@@ -22,7 +22,7 @@ class PrAuthorStatsService constructor(
     private val pullRequestStatsRepo: PullRequestStatsRepo,
     private val issueSearchPager: IssueSearchPagerService,
     private val appConfig: AppConfig,
-    private val errorProcessor: ErrorProcessor
+    private val errorProcessor: ErrorProcessor,
 ) {
 
     /**
@@ -39,7 +39,7 @@ class PrAuthorStatsService constructor(
      * ```
      */
     suspend fun authorStats(
-        prAuthorUserId: String
+        prAuthorUserId: String,
     ): AuthorStats {
         val (repoOwner, repoId, _, dateLimitAfter, dateLimitBefore) = appConfig.get()
 
@@ -50,8 +50,8 @@ class PrAuthorStatsService constructor(
                 repoId = repoId,
                 author = prAuthorUserId,
                 dateAfter = dateLimitAfter,
-                dateBefore = dateLimitBefore
-            ).toQuery()
+                dateBefore = dateLimitBefore,
+            ).toQuery(),
         ).filter {
             // Makes sure it is a PR, not an issue
             it.pull_request != null
@@ -72,7 +72,7 @@ class PrAuthorStatsService constructor(
                     pullRequestStatsRepo.stats(
                         repoOwner = repoOwner,
                         repoId = repoId,
-                        prNumber = pr.number
+                        prNumber = pr.number,
                     )
                 } catch (e: Exception) {
                     val error = errorProcessor.getDetailedError(e)
@@ -90,7 +90,7 @@ class PrAuthorStatsService constructor(
         val authorPrStats = aggregatePrAuthorsPrStats(prAuthorUserId, allMergedPrsByAuthor, mergedPrsStatsList)
         Log.i(
             "ℹ️ The author '$prAuthorUserId' has created ${authorPrStats.totalPrsCreated} PRs that successfully got merged." +
-                "\nTotal Comments Received - Code Review: ${authorPrStats.totalCodeReviewComments}, PR Comment: ${authorPrStats.totalIssueComments}, Review+Re-review: ${authorPrStats.totalPrSubmissionComments}"
+                "\nTotal Comments Received - Code Review: ${authorPrStats.totalCodeReviewComments}, PR Comment: ${authorPrStats.totalIssueComments}, Review+Re-review: ${authorPrStats.totalPrSubmissionComments}",
         )
 
         val authorReviewStats: List<AuthorReviewStats> =
@@ -103,7 +103,7 @@ class PrAuthorStatsService constructor(
     private fun aggregatePrAuthorReviewStats(
         mergedPrsStatsList: List<PrStats>,
         repoId: String,
-        prAuthorUsedId: String
+        prAuthorUsedId: String,
     ): List<AuthorReviewStats> {
         // Builds a map of reviewer ID to list PRs they have reviewed for the PR-Author
         val userReviews = mutableMapOf<UserId, List<ReviewStats>>()
@@ -117,7 +117,7 @@ class PrAuthorStatsService constructor(
                         initialResponseTime = stats.initialResponseTime[userId] ?: time,
                         prComments = stats.comments[userId] ?: noComments(userId),
                         prReadyOn = stats.prReadyOn,
-                        prMergedOn = stats.prMergedOn
+                        prMergedOn = stats.prMergedOn,
                     )
                     if (userReviews.containsKey(userId)) {
                         userReviews[userId] = userReviews[userId]!!.plus(reviewStats)
@@ -142,7 +142,7 @@ class PrAuthorStatsService constructor(
                 average = averageReviewTime,
                 totalReviews = totalReviews,
                 totalComments = totalReviewComments,
-                stats = reviewStats
+                stats = reviewStats,
             )
         }.sortedByDescending { it.totalReviews }
 
@@ -160,7 +160,7 @@ class PrAuthorStatsService constructor(
     private fun aggregatePrAuthorsPrStats(
         prAuthorUserId: String,
         allMergedPrsByAuthor: List<Issue>,
-        mergedPrsStatsList: List<PrStats>
+        mergedPrsStatsList: List<PrStats>,
     ): AuthorPrStats {
         // Builds author's stats for all PRs made by the author
         val totalPrsCreated = allMergedPrsByAuthor.size
@@ -185,7 +185,7 @@ class PrAuthorStatsService constructor(
             totalPrsCreated = totalPrsCreated,
             totalIssueComments = totalIssueComments,
             totalPrSubmissionComments = totalPrSubmissionComments,
-            totalCodeReviewComments = totalCodeReviewComments
+            totalCodeReviewComments = totalCodeReviewComments,
         )
     }
 }

src/main/kotlin/dev/hossain/githubstats/PrReviewerStatsService.kt

@@ -22,10 +22,10 @@ class PrReviewerStatsService constructor(
     private val pullRequestStatsRepo: PullRequestStatsRepo,
     private val issueSearchPager: IssueSearchPagerService,
     private val appConfig: AppConfig,
-    private val errorProcessor: ErrorProcessor
+    private val errorProcessor: ErrorProcessor,
 ) {
     suspend fun reviewerStats(
-        prReviewerUserId: String
+        prReviewerUserId: String,
     ): ReviewerReviewStats {
         val (repoOwner, repoId, _, dateLimitAfter, dateLimitBefore) = appConfig.get()
 
@@ -36,8 +36,8 @@ class PrReviewerStatsService constructor(
                 repoId = repoId,
                 reviewer = prReviewerUserId,
                 dateAfter = dateLimitAfter,
-                dateBefore = dateLimitBefore
-            ).toQuery()
+                dateBefore = dateLimitBefore,
+            ).toQuery(),
         ).filter {
             // Makes sure it is a PR, not an issue
             it.pull_request != null
@@ -58,7 +58,7 @@ class PrReviewerStatsService constructor(
                     pullRequestStatsRepo.stats(
                         repoOwner = repoOwner,
                         repoId = repoId,
-                        prNumber = pr.number
+                        prNumber = pr.number,
                     )
                 } catch (e: Exception) {
                     val error = errorProcessor.getDetailedError(e)
@@ -88,7 +88,7 @@ class PrReviewerStatsService constructor(
                     initialResponseTime = stats.initialResponseTime[prReviewerUserId] ?: prApprovalTime,
                     prComments = stats.comments[prReviewerUserId] ?: noComments(prReviewerUserId),
                     prReadyOn = stats.prReadyOn,
-                    prMergedOn = stats.prMergedOn
+                    prMergedOn = stats.prMergedOn,
                 )
             }
 
@@ -126,7 +126,7 @@ class PrReviewerStatsService constructor(
             },
             totalReviews = reviewerPrReviewStatsList.size,
             reviewedPrStats = reviewerPrReviewStatsList,
-            reviewedForPrStats = reviewerReviewedFor
+            reviewedForPrStats = reviewerReviewedFor,
         )
     }
 }

src/main/kotlin/dev/hossain/githubstats/ReviewStats.kt

@@ -45,7 +45,7 @@ data class PrStats(
     /**
      * Date and time when the PR was merged successfully.
      */
-    val prMergedOn: Instant
+    val prMergedOn: Instant,
 )
 
 /**
@@ -86,15 +86,15 @@ data class ReviewStats constructor(
     /**
      * Date and time when the PR was merged successfully.
      */
-    val prMergedOn: Instant
+    val prMergedOn: Instant,
 )
 
 /**
  * Class containing different type of stats for PR Author.
  */
 data class AuthorStats(
     val prStats: AuthorPrStats,
-    val reviewStats: List<AuthorReviewStats>
+    val reviewStats: List<AuthorReviewStats>,
 )
 
 /**
@@ -114,7 +114,7 @@ data class AuthorReviewStats(
     val average: Duration,
     val totalReviews: Int,
     val totalComments: Int,
-    val stats: List<ReviewStats>
+    val stats: List<ReviewStats>,
 )
 
 /**
@@ -128,7 +128,7 @@ data class AuthorPrStats(
     val totalPrsCreated: Int,
     val totalIssueComments: Int,
     val totalPrSubmissionComments: Int,
-    val totalCodeReviewComments: Int
+    val totalCodeReviewComments: Int,
 ) {
     /**
      * Checks if stats is empty, then it's likely not worth showing.
@@ -187,7 +187,7 @@ data class ReviewerReviewStats(
      * - john -> [PR#112 Stats, PR#931 Stats] (Meaning: The reviewer has reviewed 2 PRs created by `john`)
      * - kirk -> [PR#341 Stats, PR#611 Stats, PR#839 Stats]  (Meaning: The reviewer has reviewed 3 PRs created by `kirk`)
      */
-    val reviewedForPrStats: Map<UserId, List<PrStats>>
+    val reviewedForPrStats: Map<UserId, List<PrStats>>,
 )
 
 /**
@@ -213,7 +213,7 @@ data class UserPrComment(
      * Total PR review comment that either is [ReviewState.COMMENTED] or [ReviewState.CHANGE_REQUESTED] which
      * is used when reviewer submits a review after reviewing the PR.
      */
-    val prReviewSubmissionComment: Int
+    val prReviewSubmissionComment: Int,
 ) {
     companion object {
         /**
@@ -223,7 +223,7 @@ data class UserPrComment(
             user = userId,
             issueComment = 0,
             codeReviewComment = 0,
-            prReviewSubmissionComment = 0
+            prReviewSubmissionComment = 0,
         )
     }
 

src/main/kotlin/dev/hossain/githubstats/di/Koin.kt

@@ -40,35 +40,35 @@ val appModule = module {
         PullRequestStatsRepoImpl(
             githubApiService = get(),
             timelinesPager = get(),
-            userTimeZone = get()
+            userTimeZone = get(),
         )
     }
     factory {
         IssueSearchPagerService(
             githubApiService = get(),
-            errorProcessor = get()
+            errorProcessor = get(),
         )
     }
     factory {
         TimelineEventsPagerService(
             githubApiService = get(),
-            errorProcessor = get()
+            errorProcessor = get(),
         )
     }
     factory {
         PrReviewerStatsService(
             pullRequestStatsRepo = get(),
             issueSearchPager = get(),
             appConfig = get(),
-            errorProcessor = get()
+            errorProcessor = get(),
         )
     }
     factory {
         PrAuthorStatsService(
             pullRequestStatsRepo = get(),
             issueSearchPager = get(),
             appConfig = get(),
-            errorProcessor = get()
+            errorProcessor = get(),
         )
     }
     single { ErrorProcessor() }
@@ -80,7 +80,7 @@ val appModule = module {
             prAuthorStatsService = get(),
             resources = get(),
             appConfig = get(),
-            formatters = getAll()
+            formatters = getAll(),
         )
     }