Copilot commented on code in PR #1992: URL: https://github.com/apache/auron/pull/1992#discussion_r2780441798
########## README.md: ########## @@ -58,7 +58,7 @@ We recommend using [rustup](https://rustup.rs/) for installation. 2. Install JDK -Auron has been well tested with JDK 8, 11, and 17. +Auron has been well tested with JDK 8, 11, 17 and 21. Review Comment: The JDK list here drops the Oxford comma used elsewhere in the README (e.g., “memory management, …, and adaptive execution …”). Consider using consistent punctuation (“8, 11, 17, and 21”). Also, this statement now conflicts with CONTRIBUTING.md’s prerequisites (which still list 8/11/17), so either align the wording here (e.g., clarify ‘tested on’ vs ‘required’) or update the other doc in the same PR to avoid mixed guidance. ```suggestion Auron is regularly tested on JDK 8, 11, and 17, and is also tested on JDK 21. ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
