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.

Sign up for GitHub

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

Jump to bottom

[FEAT!] Set Up Postgres For All Environments #9

Merged
merged 4 commits into from
Feb 20, 2024
Merged

[FEAT!] Set Up Postgres For All Environments #9

merged 4 commits into from
Feb 20, 2024

Conversation

Connor-Bernard
Copy link
Member

General Info

Changes

  • Removed MySql
  • Added PG
  • Configured config/database.yml to support pg instead of MySql
    • Configured it to use environment variables
  • Set up yarn in repo to have a default yarn dev target
  • Added Makefile to help the process of standing up the new app
  • Added Overmind binary to bin and versioned in bin.lock

Testing

All changes were config; nothing to test.

Documentation

Yes. Documentation has been added to README.

Checklist

  • Name of branch corresponds to story

@Connor-Bernard Connor-Bernard self-assigned this Feb 20, 2024
@Connor-Bernard Connor-Bernard changed the title [CONFIG] Set Up Postgres For All Environments [FEAT!] Set Up Postgres For All Environments Feb 20, 2024
cynthia-lixinyi
cynthia-lixinyi approved these changes Feb 20, 2024
View reviewed changes
Copy link

@cynthia-lixinyi cynthia-lixinyi left a comment

Choose a reason for hiding this comment

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

Looks good to me

@cynthia-lixinyi cynthia-lixinyi merged commit d4fda76 into main Feb 20, 2024
2 checks passed
@Connor-Bernard Connor-Bernard deleted the 187042246-dev-pg-config branch February 20, 2024 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cynthia-lixinyi cynthia-lixinyi cynthia-lixinyi approved these changes

Assignees

@Connor-Bernard Connor-Bernard

Labels
BREAKING
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Connor-Bernard @cynthia-lixinyi