tiagosune commented on code in PR #5701:
URL: https://github.com/apache/fineract/pull/5701#discussion_r3012942921


##########
README.md:
##########
@@ -59,20 +59,23 @@ Follow these steps to quickly set up and run Apache 
Fineract locally:
 
 ### Prerequisites
 - Java 21 or higher
-- PostgreSQL or MariaDB running locally
+- PostgreSQL running locally
 
 ### Clone the repository
 ```bash
 git clone https://github.com/apache/fineract.git
 cd fineract
 ```
 
-### Create databases
+### Database setup (PostgreSQL)
+
 ```bash
-./gradlew createDB -PdbName=fineract_tenants
-./gradlew createDB -PdbName=fineract_default
+./gradlew :fineract-provider:createPGDB -PdbName=fineract_tenants
+./gradlew :fineract-provider:createPGDB -PdbName=fineract_default
 ```
 
+> Note: This task requires PostgreSQL to be properly configured. You may need 
to set environment variables such as FINERACT_HIKARI_USERNAME and 
FINERACT_HIKARI_PASSWORD. The default configuration used in CI is root / 
postgres. Additionally, an initial database (e.g. root) may need to exist for 
the connection to succeed.

Review Comment:
   > > Additionally, an initial database (e.g. root) may need to exist for the 
connection to succeed.
   > 
   > Really? And it's not there by default? That's news to me. Please confirm 
this or remove this sentence.
   
   Thanks for the feedback!
   
   I updated the commands to use the shorter task name (createPGDB) throughout 
the README.
   
   I also removed the note about the initial database, since it does not seem 
to be required by default.
   
   Please let me know if anything else should be adjusted.



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