-
Notifications
You must be signed in to change notification settings - Fork 16
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): add Docker Compose file generation #31
base: main
Are you sure you want to change the base?
feat(docker-compose): add Docker Compose file generation #31
Conversation
Signed-off-by: David Anyatonwu <davidanyatonwu@gmail.com>
cad3123
to
5fbbc5f
Compare
@onyedikachi-david thanks, have you tested this, I get this error: Where do you handle validation for docker-compose.yml file? Also please could you please implement toggle selector "With Docker Compose": |
@onyedikachi-david any updates? |
I'll making a push today. |
Signed-off-by: David Anyatonwu <davidanyatonwu@gmail.com>
@nkkko please check if it works on your end. Thanks. |
Signed-off-by: David Anyatonwu <davidanyatonwu@gmail.com>
@onyedikachi-david I’ve updated main with some changes that affect this PR. Could you please rebase your branch onto the latest main and resolve any conflicts that come up? Once that’s done, we can review the updated changes and move forward. |
@onyedikachi-david getting this error. |
Also please share screenshots or screenrecording of your solution here. |
/claim #25
Fixes: #25
Docker Compose Generation Feature
This PR adds the ability to generate Docker Compose files alongside DevContainer configurations.
Changes
DevContainerModel
inschemas.py
to include Docker Compose configurationgenerate_devcontainer_json
function inhelpers/devcontainer_helpers.py
to handle Docker Compose generation/generate
endpoint inmain.py
to return both devcontainer.json and docker-compose.yml