-
-
Notifications
You must be signed in to change notification settings - Fork 26.6k
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
Java 17: Update switch expressions #2376
Comments
Hello @iluwatar, I haven't contributed to open source yet. But I want to start now. I am:
I could take this Issue and refactor all switch expressions to the new syntax. |
Hello @iluwatar thank you for assigne me to the issue. when i checkout fresh master branch and run mvn checkstyle:check then multiple errors occure: I already have made all necressary changes to the switch statements and opened an pull request in draft. The Pull Request shows "all checks have failed" because of the checkstyle errors from master. I would be grateful if you could tell me how I should proceed here. |
@AlexMitDemBart that command is not taking into account the checkstyle configuration we have in the pom.xml. Try |
Is this something that i can fix? |
This issue has been automatically marked as stale because it has not had recent activity. The issue will be unassigned if no further activity occurs. Thank you for your contributions. |
Hey @iluwatar can I work on this issue, I am new to open source but I have a good grip in Java |
Made the pull request (issue #2376), changed all the switch expressions according to JAVA 17. |
'm new to open-sourcing and I have no clue about what I should be doing. Can anyone guide me? I have basic knowledge of java |
Java's switch expression syntax has vastly evolved over the years. It's well described at https://medium.com/@javatechie/the-evolution-of-switch-statement-from-java-7-to-java-17-4b5eee8d29b7
Acceptance criteria
The text was updated successfully, but these errors were encountered: