Skip to content
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

Include stack service configs in service specs #135

Merged
merged 1 commit into from
May 30, 2017

Conversation

johnstep
Copy link
Contributor

Signed-off-by: John Stephens johnstep@docker.com

@johnstep johnstep added this to the 17.06.0 milestone May 27, 2017
@johnstep johnstep requested a review from cpuguy83 May 27, 2017 04:12
@johnstep johnstep mentioned this pull request May 27, 2017
23 tasks
Signed-off-by: John Stephens <johnstep@docker.com>
@codecov-io
Copy link

codecov-io commented May 27, 2017

Codecov Report

Merging #135 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #135      +/-   ##
==========================================
- Coverage   44.82%   44.82%   -0.01%     
==========================================
  Files         169      169              
  Lines       11360    11361       +1     
==========================================
  Hits         5092     5092              
- Misses       5978     5979       +1     
  Partials      290      290

@johnstep johnstep changed the title Include stack service configs in the service spec Include stack service configs in service specs May 27, 2017
@vdemeester vdemeester requested a review from dnephin May 29, 2017 02:25
Copy link
Contributor

@dnephin dnephin left a comment

Choose a reason for hiding this comment

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

LGTM

@cpuguy83
Copy link
Collaborator

Do we have a unit test for convertService?

@dnephin
Copy link
Contributor

dnephin commented May 30, 2017

There isn't currently, but it should be possible to write one

Copy link
Collaborator

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

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

LGTM 🐮

Copy link
Collaborator

@cpuguy83 cpuguy83 left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants