Skip to content

add jacoco to PR - #211

Merged
wakingrufus merged 1 commit into
mainfrom
jacoco-action
Sep 17, 2026
Merged

wakingrufus merged 1 commit into
mainfrom
jacoco-action

Conversation

@wakingrufus

@wakingrufus wakingrufus commented Sep 17, 2026 •

Copy link
Copy Markdown
Member

add jacoco to PR

@github-actions

github-actions Bot commented Sep 17, 2026 •

Copy link
Copy Markdown

ArchRules report

@wakingrufus wakingrufus changed the title support apply false in plugins dsl add jacoco to PR Sep 17, 2026
@github-actions

github-actions Bot commented Sep 17, 2026 •

Copy link
Copy Markdown

ArchRules report

@wakingrufus
wakingrufus marked this pull request as ready for review September 17, 2026 14:42
@github-actions

github-actions Bot commented Sep 17, 2026 •

Copy link
Copy Markdown

ArchRules report

@wakingrufus
wakingrufus force-pushed the jacoco-action branch 2 times, most recently from d58c299 to e448ae1 Compare September 17, 2026 14:49
@github-actions

Copy link
Copy Markdown

There is no coverage information present for the Files changed

@github-actions

github-actions Bot commented Sep 17, 2026 •

Copy link
Copy Markdown

ArchRules report

@github-actions

github-actions Bot commented Sep 17, 2026 •

Copy link
Copy Markdown

ArchRules report

@wakingrufus
wakingrufus force-pushed the jacoco-action branch 2 times, most recently from bfdb686 to 0a5aacd Compare September 17, 2026 15:17
@github-actions

github-actions Bot commented Sep 17, 2026 •

Copy link
Copy Markdown

ArchRules report

convert PluginsBuilder to show coverage
@github-actions

github-actions Bot commented Sep 17, 2026 •

Copy link
Copy Markdown

JaCoCo Code Coverage

Overall Project 79.58% ❌
Files changed 100% 🍏

File Coverage
PluginsBuilder.kt 100% 🍏
Plugin.java 100% 🍏

@wakingrufus
wakingrufus merged commit f91ffa9 into main Sep 17, 2026
3 of 4 checks passed
@wakingrufus
wakingrufus deleted the jacoco-action branch September 17, 2026 15:42

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

converted so jacoco will actually see changes (and we want to convert these to kotlin anyway)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant