chore: upgrade pull-request workflow from @v2 to @main#41
Merged
morten-andersen merged 5 commits intomainfrom Feb 11, 2026
Merged
chore: upgrade pull-request workflow from @v2 to @main#41morten-andersen merged 5 commits intomainfrom
morten-andersen merged 5 commits intomainfrom
Conversation
Updates pull-request.yaml@v2 → pull-request-kotlin.yml@main. Adds SONAR_TOKEN to enable SonarCloud analysis. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Test Results123 tests 123 ✅ 10s ⏱️ Results for commit 465d579. ♻️ This comment has been updated with latest results. |
morten-andersen
approved these changes
Feb 10, 2026
morten-andersen
requested changes
Feb 10, 2026
Contributor
morten-andersen
left a comment
morten-andersen
left a comment
There was a problem hiding this comment.
Kover is not set up for this lib
Member
Author
|
Perhaps i should add kover coverage here then ? |
Adds the Kover plugin to enable code coverage reporting, required by the upgraded pull-request-kotlin.yml@main workflow. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Fixes dependency resolution issue where Kover couldn't resolve kotlin-bom from the root project's koverExternalArtifacts config. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Points to core module's coverage report since Kover runs per-submodule rather than at the root. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Code Coverage
|
morten-andersen
approved these changes
Feb 11, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Updates
pull-request.yaml@v2→pull-request-kotlin.yml@mainin github-workflows.Changes:
pull-request.yamlwas renamed/replaced bypull-request-kotlin.ymlon main (uses Kover for coverage instead of Jacoco)SONAR_TOKENto enable SonarCloud analysis (org secret, previously not configured)Other improvements on main: Java 21 default, gradle multi-module support, ktlint, test result publishing, PR title check.
Test plan
🤖 Generated with Claude Code