diff options
author | hannibal2 <24389977+hannibal00212@users.noreply.github.com> | 2024-03-13 21:19:28 +0100 |
---|---|---|
committer | hannibal2 <24389977+hannibal00212@users.noreply.github.com> | 2024-03-13 21:19:28 +0100 |
commit | bdd92b4f440063f582a242670afd38d61b76f6d1 (patch) | |
tree | e77152f8350d1be98a23d9d71eab47c651f3ec5e /src/main/kotlin | |
parent | 4161d7696fe1c86f9d53c68c7ffddf122e5c7578 (diff) | |
download | SkyHanniChangelogBuilder-bdd92b4f440063f582a242670afd38d61b76f6d1.tar.gz SkyHanniChangelogBuilder-bdd92b4f440063f582a242670afd38d61b76f6d1.tar.bz2 SkyHanniChangelogBuilder-bdd92b4f440063f582a242670afd38d61b76f6d1.zip |
warn when wrong pr name
Diffstat (limited to 'src/main/kotlin')
-rw-r--r-- | src/main/kotlin/Main.kt | 49 |
1 files changed, 41 insertions, 8 deletions
diff --git a/src/main/kotlin/Main.kt b/src/main/kotlin/Main.kt index b26c52d..e4aadb7 100644 --- a/src/main/kotlin/Main.kt +++ b/src/main/kotlin/Main.kt @@ -9,12 +9,12 @@ import java.util.regex.Matcher import java.util.regex.Pattern import kotlin.system.exitProcess -enum class Category(val changeLogName: String) { - NEW("New Features"), - IMPROVEMENT("Improvements"), - FIX("Fixes"), - INTERNAL("Technical Details"), - REMOVED("Removed Features"), +enum class Category(val changeLogName: String, val prTitle: String) { + NEW("New Features", "Feature"), + IMPROVEMENT("Improvements", "Improvement"), + FIX("Fixes", "Fix"), + INTERNAL("Technical Details", "Backend"), + REMOVED("Removed Features", "Removed Feature"), ; } @@ -78,6 +78,7 @@ fun readPrs( var errors = 0 var excluded = 0 var done = 0 + var wrongPrName = 0 // TODO find better solution for this sorting logic val filtered = when (whatToDo) { WhatToDo.NEXT_BETA -> prs.filter { it.mergedAt != null } @@ -96,6 +97,7 @@ fun readPrs( val number = pr.number val prLink = pr.htmlUrl val body = pr.body + val title = pr.title val description = body?.split(System.lineSeparator()) ?: emptyList() if (description.isNotEmpty()) { @@ -108,7 +110,11 @@ fun readPrs( } } try { - allChanges.addAll(parseChanges(description, prLink, categories)) + val newChanges = parseChanges(description, prLink) + if (hasWrongPrName(prLink, title, newChanges)) { + wrongPrName++ + } + allChanges.addAll(newChanges) done++ } catch (t: Throwable) { println("") @@ -130,7 +136,10 @@ fun readPrs( println("Excluded $excluded PRs.") } if (errors > 0) { - println("Found $errors PRs with errors.") + println("Found $errors PRs with errors!") + } + if (wrongPrName > 0) { + println("Found $wrongPrName PRs with wrong names!") } println("Loaded $done PRs correctly.") if (errors > 0) { @@ -140,6 +149,30 @@ fun readPrs( } } +fun hasWrongPrName(prLink: String, title: String, newChanges: List<Change>): Boolean { + val hasFix = newChanges.any { it.category == Category.FIX } + for (category in Category.entries) { + if (newChanges.any { it.category == category }) { + var start = category.prTitle + if (hasFix && category != Category.FIX) { + start += " + Fix" + } + start += ": " + val wrongName = !title.startsWith(start) + if (wrongName) { + println("wrong pr title!") + println("found: '$title'") + println("should start with $start") + println("link: $prLink") + println(" ") + } + return wrongName + } + } + + return false +} + enum class OutputType { DISCORD_INTERNAL, GITHUB, DISCORD_PUBLIC, } |