Skip to content
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

DTSPO-18332 - Rename aso postgres server script and update documentation #5975

Merged
merged 1 commit into from
Jan 21, 2025

Conversation

louisehuyton
Copy link
Contributor

@louisehuyton louisehuyton commented Jan 21, 2025

Jira link

DTSPO-18332

Change description

  • Rename postgres-setup.sh to 'add-postgres-server.sh'
  • Update references to script ^^
  • Include example output from script in documentation

Testing done

N/A

Checklist

  • commit messages are meaningful and follow good commit message guidelines
  • README and other documentation has been updated / added (if needed)
  • tests have been updated / new tests has been added (if needed)
  • Does this PR introduce a breaking change

🤖AEP PR SUMMARY🤖

Docs/aso-setup-v2.md

  • Updated the documentation to mention running add-postgres-server.sh instead of postgres-setup.sh for setting up the Postgres server.
  • Added a note to refer to a PR for expected output of the script.

bin/v2/postgres-setup.sh

  • File renamed from postgres-setup.sh to add-postgres-server.sh.
  • Updated the usage function to reflect the new filename add-postgres-server.sh.

…es and include example output in documentation
@louisehuyton louisehuyton requested review from a team, endakelly, cpareek, Tyler-35, dawidstrozak, JordanHoey96, ieuanb74 and ryanmcalary11 and removed request for a team January 21, 2025 12:51
@louisehuyton louisehuyton requested a review from a team as a code owner January 21, 2025 12:51
@louisehuyton louisehuyton merged commit 3e56fae into master Jan 21, 2025
18 checks passed
@louisehuyton louisehuyton deleted the DTSPO-18332_UpdateDocs branch January 21, 2025 13:00
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