Skip to content

Conversation

mtfoley
Copy link

@mtfoley mtfoley commented Sep 15, 2023

Description

Adds a docker compose file to ease local development.

  • Docker compose spins up local postgres database
  • Docker compose includes migrate service that prepares database for use by API
  • Docker compose includes service that allows hot reloading of microservice, e.g. using "air" utility

What type of PR is this? (check all applicable)

  • 🍕 Feature
  • 🐛 Bug Fix
  • 📝 Documentation Update
  • 🎨 Style
  • 🧑‍💻 Code Refactor
  • 🔥 Performance Improvements
  • ✅ Test
  • 🤖 Build
  • 🔁 CI
  • 📦 Chore (Release)
  • ⏩ Revert

Related Tickets & Documents

Resolves #50

Mobile & Desktop Screenshots/Recordings

Added tests?

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help

Added to documentation?

  • 📜 README.md
  • 📓 docs.opensauced.pizza
  • 🍕 dev.to/opensauced
  • 📕 storybook
  • 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

…r-compose

- makes adjustments for upstream sslmode-disable flag
- consolidates some commands in Migrate.Dockerfile
@jpmcb
Copy link
Member

jpmcb commented Nov 21, 2023

Hi @mtfoley !! Sorry for the long turn around on this: I've been heads down on some stuff. Anywhere I can help with this?

@mtfoley mtfoley changed the title Docker compose feat: docker compose files Nov 22, 2023
@mtfoley
Copy link
Author

mtfoley commented Nov 22, 2023

Hey @jpmcb not specifically other than to let me know if you see anything that needs adjustment thus far. I saw the team had added a SQL file not long ago to handle migrations with, and the Migrate.Dockerfile in this PR is trying to piece something similar together on the fly, so there's probably some reconciling to do there.

The only other thing I was going to look at implementing was the air utility for Go in the app service and then just adding things to the README.

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.

Full stack docker compose file for local dev

2 participants