-
Notifications
You must be signed in to change notification settings - Fork 78
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
FR/RFC: Docker Compose enhancements #229
Comments
FWIW, for general editing (e.g. completions) in I've added an entry to (use-package lsp-mode
:mode "docker-compose.yml"
;; ....
) |
Adding meqif/docker-compose-mode to OP. |
Sorry I completely missed this issue. It sounds nice, but with my limited time I'm a bit afraid of the complexity of maintaining a docker-compose mode. Also, maybe design wise I'm not sure if this belongs to docker.el or to docker-compose.el. About automatically changing to the directory root, this should already happen if you start the transient from the docker-compose.yml file... tho what you describe seems to hint at some projectile-docker-compose mode or smth? |
Eminently reasonable. Sounds fair enough.
I was also concerned about this at first and I was leaning in doing it in
These had me 50/50, specially the last one.
Yup. CWD should change to run DC. Maybe just at the last second, just before calling DC. |
docker.el
is totally awesome, thanks for sharing.While working with
docker-compose.yml
files, I've found that no specific actions/mode is enabled. Some enhancements/ideas I'd like to propose and, with proper guidance and if they're in my knowledge range, happy to tackle sometime.docker-compose-mode
to allow more granular key bindings.docker-mode
fordocker-compose.yml
files.C
ompose prefix the root/default when invoking the transient fordocker-compose
mode.Afterwards, more workflow improvements might be possible to code/document for the hack->compose-up->repeat workflow.
Dumping the idea, so far 😄
EDIT: Turns out there is already a docker-compose-mode, albeit it seems inactive. Maybe it can be of some help.
The text was updated successfully, but these errors were encountered: