Skip to content

Add section about PostgreSQL dependency #4376

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

Merged
merged 5 commits into from
Jun 20, 2025

Conversation

MatthewVaadin
Copy link
Contributor

Adds a section that describes how to add the PostgreSQL dependency to an application.

Part of https://github.com/vaadin/control-center/issues/964

@MatthewVaadin MatthewVaadin self-assigned this Jun 16, 2025
Copy link

github-actions bot commented Jun 16, 2025

AI Language Review

For the file flyway-migrations:

  • Improve the phrase "a simple way to set up" to "an easy method for setting up" for better readability.
  • In the note section, consider rephrasing “NOTE: The Database feature” to “Note: The database feature” to maintain consistency with the general writing style.

For the file database:

  • In the opening sentence, change “a simple way to provision” to “an easy method for provisioning” to enhance clarity.
  • The phrase “the PostgreSQL JDBC driver to be included” should be revised to “the PostgreSQL JDBC driver needs to be included” for grammatical accuracy.

@MatthewVaadin MatthewVaadin requested a review from heruan June 16, 2025 12:37
@heruan heruan enabled auto-merge (squash) June 18, 2025 14:29
@heruan heruan merged commit 134da21 into latest Jun 20, 2025
4 of 5 checks passed
@heruan heruan deleted the control-center/964/postgres-dependency branch June 20, 2025 14:05
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.

2 participants