Skip to content

Fix NPE in KotlinSerializableFilter #1970

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.

Sign up for GitHub

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

Jump to bottom
Merged
Godin merged 5 commits into from
Oct 9, 2025
Merged

Fix NPE in KotlinSerializableFilter #1970

Godin merged 5 commits into from
Oct 9, 2025
+168 −1

Conversation

@Godin
Copy link
Member

@Godin Godin commented Oct 8, 2025 •
edited
Loading

This was overlooked in #1885
Fixes #1969

@Godin Godin added this to the 0.8.14 milestone Oct 8, 2025
@Godin Godin self-assigned this Oct 8, 2025
@Godin Godin added this to Filtering Oct 8, 2025
@github-project-automation github-project-automation bot moved this to Awaiting triage in Filtering Oct 8, 2025
@Godin Godin force-pushed the KotlinSerializableFilter branch from 581e886 to 2d76d5c Compare October 8, 2025 13:15
@Godin Godin moved this from Awaiting triage to In Progress in Filtering Oct 8, 2025
@Godin Godin mentioned this pull request Oct 8, 2025
Closed
@Godin Godin force-pushed the KotlinSerializableFilter branch from 88447bb to 157e916 Compare October 8, 2025 21:55
@Godin Godin force-pushed the KotlinSerializableFilter branch from 157e916 to 4c120e5 Compare October 9, 2025 12:11
@Godin Godin marked this pull request as ready for review October 9, 2025 13:53
@Godin Godin requested a review from leveretka October 9, 2025 13:53
@Godin Godin merged commit 89c4bd5 into jacoco:master Oct 9, 2025
36 checks passed
@Godin Godin deleted the KotlinSerializableFilter branch October 9, 2025 15:34
@github-project-automation github-project-automation bot moved this from In Progress to Done in Filtering Oct 9, 2025
@marcin-arciszewski-gds marcin-arciszewski-gds mentioned this pull request Oct 17, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@leveretka leveretka leveretka approved these changes

Assignees

@Godin Godin

Labels

component: core.filters language: Kotlin type: bug 🐛 Something isn't working

Projects

Filtering
Status: Done

Milestone

0.8.14

Development

Successfully merging this pull request may close these issues.

KotlinSerializationFilter does not work with sealed classes annotated with @Serializable and crashes JaCoCo report generation

2 participants

@Godin @leveretka