Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

✅ Remove @Composable from Kover reports #856

Merged
merged 1 commit into from
Feb 17, 2025

Conversation

LeoColman
Copy link
Owner

Kover doesn't support integration tests anyway, so keeping Composable (ie integration-test) functions under kover reports is not helpful.

This commit makes our test coverage reflect a value closer to reality for our unit tests

@LeoColman LeoColman enabled auto-merge (rebase) February 17, 2025 22:38
@LeoColman LeoColman disabled auto-merge February 17, 2025 22:40
Kover doesn't support integration tests anyway, so keeping Composable
(ie integration-test) functions under kover reports is not helpful.

This commit makes our test coverage reflect a value closer to reality
for our unit tests

Signed-off-by: Leonardo Colman Lopes <[email protected]>
@LeoColman LeoColman force-pushed the chore/kover-remove-composables branch from 37ec263 to 98cc717 Compare February 17, 2025 22:46
@LeoColman LeoColman enabled auto-merge (rebase) February 17, 2025 22:47
@LeoColman LeoColman merged commit 4a68494 into main Feb 17, 2025
10 checks passed
@LeoColman LeoColman deleted the chore/kover-remove-composables branch February 17, 2025 22:52
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