Skip validation when extracting config variables#13831
Conversation
Signed-off-by: bigmomma <scarab.systems@yahoo.com>
|
/review |
|
❌ PR Review Failed — The review agent encountered an error and could not complete the review. View logs. |
glours
left a comment
There was a problem hiding this comment.
The fix looks correct, but I’d like us to add a regression test covering the up path as well.
The issue originally affects the remote-stack prompt flow used by up, so having coverage there would help ensure this does not regress again.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
Thanks for the review. That makes sense, and I appreciate the context on the |
Signed-off-by: bigmomma <scarab.systems@yahoo.com>
|
Added the focused |
What I did
config --variablesand remote-stack interpolation-variable prompting.host_ipandpublished.Related issue
Fixes #13613
(not mandatory) A picture of a cute animal, if possible in relation to what you did

![cute fuzzy animal]
Validation
go test ./cmd/composedocker buildx bake lint