Skip to content

Conversation

@FXV6624
Copy link
Contributor

@FXV6624 FXV6624 commented Mar 3, 2025

Improved the README.md by adding:
-Compatibility with Java 11 and 17 (updated from Java 7/8). Tested using Java 17
-Quick use example to facilitate adoption. Basic example about sql2o's main functions, new users don´t need to visit the wiki or the code guidelines to see how sql2o works

@@ -1,19 +1,53 @@
# sql2o [![Github Actions Build](https://github.com/aaberg/sql2o/actions/workflows/pipeline.yml/badge.svg)](https://github.com/aaberg/sql2o/actions) [![Maven Central](https://img.shields.io/maven-central/v/org.sql2o/sql2o.svg)](https://search.maven.org/search?q=g:org.sql2o%20a:sql2o)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you remove the github actions and maven labels?

When fetching data from the database, the ResultSet will automatically be filled into your POJO objects.
Kind of like an ORM, but without the SQL generation capabilities.
Sql2o requires at Java 7 or 8 to run. Java versions past 8 may work, but is currently not supported.
Sql2o is compatible with **Java 8 and later versions**, including Java 11 and 17.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, that old description, saying that it isn't compatible with Java 8 or newer, is very misleading. Thanks for updating!

@ADjQ
Copy link

ADjQ commented Mar 9, 2025

Hi,
I just want to point out that according to the releases tab, in version 1.9.0-RC1, it specifically says "From this version, sql2o is only compatible with java 17 or newer".
I think that should be reflected in the README too, since that means the current version is not compatible with Java 11, which contradicts the new line regarding the compatibility of the library

@aaberg aaberg merged commit 01ef698 into aaberg:master Mar 9, 2025
1 of 2 checks passed
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.

3 participants