Skip to content

Made dev compose file so u don't have to rebuild everytime you make changes#77

Merged
BigSpaceships merged 2 commits intoComputerScienceHouse:mainfrom
tmcnulty387:main
Feb 18, 2026
Merged

Made dev compose file so u don't have to rebuild everytime you make changes#77
BigSpaceships merged 2 commits intoComputerScienceHouse:mainfrom
tmcnulty387:main

Conversation

@tmcnulty387
Copy link
Member

…hanges

What

Added docker-compose.dev.yaml to run the vote container during development

Why

So you don't have to rebuild the container when you change code during development

Test Plan

Just changed the docker compose file and README, didn't run Go tests. I verified that it actually works by adding print statements in main, output does change when rerunning podman compose -f docker-compose.dev.yaml up

Env Vars

Didn't change, just copied existing compose file and added this

  vote:
    build:
      context: .
      target: build
    working_dir: /src
    command: ["go", "run", "./..."]
    volumes:
      - type: bind
        source: .
        target: /src

Checklist

  • Tested all changes locally

@tmcnulty387
Copy link
Member Author

Fixes #76

@tmcnulty387
Copy link
Member Author

I'll add the other commits in a different PR, force push was a reset hard to docker compose commit

@BigSpaceships
Copy link
Contributor

I said this on slack but I love these changes but we only use the compose file for dev so would you be willing to add the bind mount to the main compose file?

@tmcnulty387
Copy link
Member Author

I said this on slack but I love these changes but we only use the compose file for dev so would you be willing to add the bind mount to the main compose file?

Sry I missed the slack msg, I made a seperate file because I wasn't sure how the bind would affect prod. I'll go ahead and make it one file.

@BigSpaceships
Copy link
Contributor

This is great, and no worries about slack I said it in the same place each time :) anyway thank you so much!

@BigSpaceships BigSpaceships merged commit a3a01ea into ComputerScienceHouse:main Feb 18, 2026
tallen42 added a commit that referenced this pull request Feb 18, 2026
Made compose file run go so you don't have to always rebuild (#77)
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

Comments