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
introduce top-level name
#206
Conversation
No, I don't think we should do this, and this definitely doesn't address docker/compose#745 (just shifts the problem from directory name or .env to the compose file) |
@thaJeztah then I misunderstood the requirements set on docker/compose#745 |
@thaJeztah do you mean because it doesn't persist the project name in a separate file from the docker-compose.yml file, and/or because compose does not save the auto-generated project name into said separate file? This solution does allow persistence of the project name independently of choices that may affect other (not compose related) parts of a project (folder name, checking in a .env file), so I think it's fair to say it covers at least some of the use cases -- I think some clarification on what it's missing would be helpful. |
@thaJeztah ok, reading more carefully your initial request docker/compose#745. IIUC you're expecting we store "user session state" on filesystem aside the compose.yaml file so that user only has to set That being said, comments on docker/compose#745 demonstrate we also have user who expect they can hard-code the project name in the compose file to make it fully deterministic. I think we can have both
|
+1 to have an option to define name as part of the compose file. Indeed, it does not need to be the only option, but in many cases defining it inside the compose file is the most intiuitive and natural solution. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tiny typo inline 💬 More general question: how do we expect it to work for cases when multiple Compose files are combined?
Like other elements : if defined in an override file, will override. |
Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
0257a0d
to
bb155bb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good (with one typo/suggestion inline) ✅
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great ✅ Very excited about this addition 🎉
What this PR does / why we need it:
introduce top-level
name
to address docker/compose#745