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

feat: docker compose files #51

Draft
wants to merge 3 commits into
base: beta
Choose a base branch
from

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