Skip to content

Conversation

mikran
Copy link

@mikran mikran commented Aug 15, 2025

"content" is not part of volumes in compose spec. I've updated example to match desired outcome "Here you can see how to add a file with content and a dynamic value that is coming from an environment variable." with configs instead.

Copy link

Hi @mikran,

This PR has been automatically closed because it has no description.
Please provide a valid description explaining your changes and reopen the PR.

@github-actions github-actions bot closed this Aug 15, 2025
@mikran
Copy link
Author

mikran commented Aug 15, 2025

I've added description, but I don't have access to reopen the PR

@peaklabs-dev peaklabs-dev reopened this Aug 15, 2025
@github-actions github-actions bot changed the base branch from v4.x to next August 15, 2025 13:36
Copy link

Hi @mikran,

The base branch of this PR has been automatically changed to next.
All contributors must commit changes to the next branch.

A maintainer will review your PR within 5 to 7 business days. Thank you!

@ShadowArcanist ShadowArcanist added ✨ Enhancement Suggestions to improve or add detail to existing docs. ⚖️ Medium Priority Important tasks to be addressed soon, but not urgent. labels Aug 15, 2025
@peaklabs-dev
Copy link
Member

It's not part of the volumes in compose spec but Coolify has added this custom content option to easily provide content in a file. Do you think it would be better to use configs option from compose instead (in the future)?

@mikran
Copy link
Author

mikran commented Aug 16, 2025

I don't know why it was added - but to avoid the confusion I would like to see the (this is not available in a normal docker-compose) comment in it, same as is_directory option before it

@peaklabs-dev
Copy link
Member

@mikran It will probably removed in the next major version. For now can you add that comment and maybe add the configs way as another example.

@peaklabs-dev peaklabs-dev added the ⏱️ PR - Waiting for Changes PR needs changes from the author before proceeding. label Aug 18, 2025
@mikran
Copy link
Author

mikran commented Aug 20, 2025

@mikran It will probably removed in the next major version. For now can you add that comment and maybe add the configs way as another example.

I added that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Enhancement Suggestions to improve or add detail to existing docs. ⚖️ Medium Priority Important tasks to be addressed soon, but not urgent. ⏱️ PR - Waiting for Changes PR needs changes from the author before proceeding.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants