Copilot commented on code in PR #2467:
URL: https://github.com/apache/age/pull/2467#discussion_r3533623342


##########
README.md:
##########
@@ -125,7 +125,7 @@ Apache AGE is intended to be simple to install and run. It 
can be installed with
  Install PostgreSQL
 </h4>
 
-You will need to install an AGE compatible version of Postgres<a>, for now AGE 
supports Postgres 11, 12, 13, 14, 15, 16, 17 & 18. Supporting the latest 
versions is on AGE roadmap.
+You will need to install an AGE compatible version of Postgres, for now AGE 
supports Postgres 11, 12, 13, 14, 15, 16, 17, 18, 19beta1. Supporting the 
latest versions is on AGE roadmap.

Review Comment:
   The README now states that PostgreSQL 19beta1 is supported, but the 
repository’s CI/installcheck workflow for this branch only builds against 
REL_18_STABLE and there are no other references to PG19 in the codebase. Unless 
PG19beta1 support is intentionally being claimed (with corresponding CI/compat 
testing), this line should not list 19beta1 as a supported version, and the 
sentence should be punctuated for clarity.



##########
README.md:
##########
@@ -152,7 +152,7 @@ You can <a href="https://www.postgresql.org/ftp/source/";> 
download the Postgres
 
 Clone the <a href="https://github.com/apache/age";>github repository</a> or 
download the <a href="https://github.com/apache/age/releases";>download an 
official release.
 </a>
-Run the pg_config utility and check the version of PostgreSQL. Currently, only 
PostgreSQL versions 11, 12, 13, 14, 15, 16, 17 & 18 are supported. If you have 
any other version of Postgres, you will need to install PostgreSQL version 11, 
12, 13, 14, 15, 16, 17 & 18.
+Run the pg_config utility and check the version of PostgreSQL. Currently, only 
PostgreSQL versions 11, 12, 13, 14, 15, 16, 17, 18, 19beta1 are supported. If 
you have any other version of PostgreSQL, you will need to install PostgreSQL 
version 11, 12, 13, 14, 15, 16, 17, 18, 19beta1.

Review Comment:
   This paragraph claims PostgreSQL 19beta1 is supported, but there’s no 
evidence of PG19 coverage in CI (e.g., installcheck.yaml only targets 
REL_18_STABLE). If PG19beta1 is not actively tested/supported, remove it from 
the supported list to avoid misleading users (and avoid recommending a beta as 
the required install target).



-- 
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]

Reply via email to