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.

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

SerialVersionUIDInSerializableClass highlights complete class when it finds warning #6109

Closed
atulgpt opened this issue May 19, 2023 · 0 comments · Fixed by #6114
Closed

SerialVersionUIDInSerializableClass highlights complete class when it finds warning #6109

atulgpt opened this issue May 19, 2023 · 0 comments · Fixed by #6114
Labels
good first issue Issue that is easy to pickup for people that are new to the project help wanted improvement

Comments

@atulgpt
Copy link
Contributor

atulgpt commented May 19, 2023

Expected Behavior

SerialVersionUIDInSerializableClass In case it finds serialVersionUID which is not private or const it should only highlight serialVersionUID

In case serialVersionUID is missing it should only highlight the class name

Observed Behavior

The whole class is highlighted. It hides other issues in that class
Screenshot 2023-05-19 at 8 25 00 PM
@atulgpt atulgpt added the bug label May 19, 2023
@BraisGabin BraisGabin added improvement help wanted good first issue Issue that is easy to pickup for people that are new to the project and removed bug labels May 20, 2023
@atulgpt atulgpt mentioned this issue May 20, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Issue that is easy to pickup for people that are new to the project help wanted improvement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

SerialVersionUIDInSerializableClass - Update the error location atulgpt/detekt
2 participants
@BraisGabin @atulgpt