-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Break java #9451
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
base: next
Are you sure you want to change the base?
Break java #9451
Conversation
Summary of ChangesHello @harshyyy21, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request implements a breaking change by enforcing a minimum Java version of 21 for running Firebase emulators. Previously, users with older Java versions would receive a warning, but the emulators might still function. With this update, any attempt to start emulators using Java versions below 21 will now trigger a critical error, preventing them from launching. This ensures that the emulator suite operates within a supported and modern Java environment. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
This pull request correctly enforces the minimum Java version requirement by throwing an error for versions below 21, instead of just showing a warning. The changes are logical and achieve the intended goal. I have two suggestions to improve this change. First, I recommend explicitly marking this as a breaking change in the changelog for better user visibility. Second, I've pointed out that the error message is logged twice, which is redundant, and suggested a way to clean this up for a better user experience. Overall, a solid update.
Description
No longer support Java versions below 21.
Scenarios Tested
Tested this with Java 11 and Java 25.
Java 11 output:
Java 25 output: