Abandon ldc#1098
Merged
Merged
Conversation
3433922 to
8efcd43
Compare
8efcd43 to
58badaa
Compare
BryonLewis
approved these changes
Jan 3, 2022
Collaborator
BryonLewis
left a comment
There was a problem hiding this comment.
pulled, built, ran integration test, ran some development testing and all seems to work well. I just need to remind myself that ./server is hotload by default all the time so if I do an up -d and start messing around I can kill the container with errors during a save.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
While I still like LDC as an idea, and I would probably prefer to use it myself, everyone who gets started using this project seems confused by it. It also protects the user from having to learn certain things about how docker-compose works, which is a drawback.
@Erotemic recently made a PR Kitware/ldc#2 that sorta forced a decision here: Keep supporting LDC as a clever 50 line shell script or just do something more familiar to Girder 4 with an override file.
I think we should abandon ldc.
In order to make the transition nice and smooth, I recommend the following shell aliases. Once these are in place, you shouldn't notice much difference from ldc
Migration guide
The behavior of
ldc up -dhas been removed.ldc dev up -dbecomes the default because of the override file. The thing I really liked aboutldcwas that it allowed you to explicitly and conveniently differentiate between "run with the code bundled at build time" and "run with code mounted at runtime". This distinction has never been as clear with vanilla docker compose, but it's probably better for users to just understand how it works than to have LDC protecting you.