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

Issue #2793 #2795

Merged
merged 2 commits into from
Jan 6, 2024
Merged

Issue #2793 #2795

merged 2 commits into from
Jan 6, 2024

Conversation

gtoison
Copy link
Contributor

@gtoison gtoison commented Jan 5, 2024

SerializableIdiom reports that a serialVersionUid is needed for a record class.
It also reports that SE_NO_SUITABLE_CONSTRUCTOR_FOR_EXTERNALIZATION for an externalizable record even if records serialization does not work with externalizable

@gtoison gtoison marked this pull request as ready for review January 5, 2024 16:35
@hazendaz hazendaz self-assigned this Jan 6, 2024
@hazendaz hazendaz merged commit ed97e1f into spotbugs:master Jan 6, 2024
9 checks passed
@hazendaz
Copy link
Member

hazendaz commented Jan 6, 2024

@gtoison Can you add change log? New PR, sorry merged as LGTM but failed to notice no change record for this.

gtoison added a commit that referenced this pull request Jan 6, 2024
@gtoison
Copy link
Contributor Author

gtoison commented Jan 6, 2024

Done here #2799

hazendaz pushed a commit that referenced this pull request Jan 6, 2024
@hazendaz hazendaz added this to the SpotBugs 4.8.4 milestone Jan 6, 2024
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.

None yet

2 participants