Skip to content
This repository was archived by the owner on Jul 18, 2025. It is now read-only.

Carry #74 Expose Interpolate for external tools#181

Merged
vdemeester merged 1 commit intodocker-archive-public:masterfrom
joshwget:expose-interpolate
Mar 27, 2016
Merged

Carry #74 Expose Interpolate for external tools#181
vdemeester merged 1 commit intodocker-archive-public:masterfrom
joshwget:expose-interpolate

Conversation

@joshwget
Copy link
Copy Markdown
Contributor

Changed a few new instances of rawService and rawServiceMap. As discussed in #99, interpolation will be moved into a config package separately from this PR.


It is quite useful to parse a text file in the same exact way that
compose does. One can imagine this being useful for GUI applications or
custom parsers.

Signed-off-by: Darren Shepherd darren@rancher.com

Conflicts:
project/merge.go

It is quite useful to parse a text file in the same exact way that
compose does.  One can imagine this being useful for GUI applications or
custom parsers.

Signed-off-by: Darren Shepherd <darren@rancher.com>

Conflicts:
	project/merge.go
type rawService map[string]interface{}
type rawServiceMap map[string]rawService
// RawService is represent a Service in map form unparsed
type RawService map[string]interface{}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When moving to a config package, I think we should use a more abstracted name 😝

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed!

@vdemeester
Copy link
Copy Markdown
Contributor

@joshwget you beat me to it 👍 😉

@vdemeester
Copy link
Copy Markdown
Contributor

LGTM 🐯

@vdemeester vdemeester merged commit e290a51 into docker-archive-public:master Mar 27, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants