Skip to content

add some optimized functions in example/deployment #10372

Merged
openshift-bot merged 1 commit intoopenshift:masterfrom
yuexiao-wang:optimize-deploy-example
Aug 15, 2016
Merged

add some optimized functions in example/deployment #10372
openshift-bot merged 1 commit intoopenshift:masterfrom
yuexiao-wang:optimize-deploy-example

Conversation

@yuexiao-wang
Copy link
Copy Markdown
Contributor

@yuexiao-wang yuexiao-wang commented Aug 12, 2016

It is poorly readable that the incoming parameters of function is so lengthiness. I add some functions to instead of lengthiness parameters which is clearly understanded

Signed-off-by: yuexiao-wang wang.yuexiao@zte.com.cn

@yuexiao-wang
Copy link
Copy Markdown
Contributor Author

@mfojtik PTAL
Thanks!

@mfojtik
Copy link
Copy Markdown
Contributor

mfojtik commented Aug 12, 2016

@yuexiao-wang looks good to me, thanks! Can you please reword the commit message and make it more descriptive?

@yuexiao-wang
Copy link
Copy Markdown
Contributor Author

Hi mfojtik,
I have reworded the commit messages.
Thanks!
@mfojtik PTAL

@yuexiao-wang yuexiao-wang force-pushed the optimize-deploy-example branch 2 times, most recently from fa1e15c to 345fcdb Compare August 12, 2016 10:10
@mfojtik
Copy link
Copy Markdown
Contributor

mfojtik commented Aug 12, 2016

@yuexiao-wang I don't see the commit message changed :-)

Signed-off-by: yuexiao-wang <wang.yuexiao@zte.com.cn>
@yuexiao-wang yuexiao-wang force-pushed the optimize-deploy-example branch from 345fcdb to 2a42d91 Compare August 13, 2016 03:22
@yuexiao-wang
Copy link
Copy Markdown
Contributor Author

yuexiao-wang commented Aug 13, 2016

@mfojtik PTAL
I' m sorry to not understand your suggestion. I modified the PR message but not changed the commit message. I have changed the commit messages just now.
Thanks!

@yuexiao-wang
Copy link
Copy Markdown
Contributor Author

@mfojtik PTAL
How about this PR? Thanks!

@mfojtik
Copy link
Copy Markdown
Contributor

mfojtik commented Aug 15, 2016

LGTM [merge]

@openshift-bot
Copy link
Copy Markdown
Contributor

openshift-bot commented Aug 15, 2016

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/7912/) (Image: devenv-rhel7_4828)

@openshift-bot
Copy link
Copy Markdown
Contributor

Evaluated for origin merge up to 2a42d91

@openshift-bot openshift-bot merged commit 45275ec into openshift:master Aug 15, 2016
@yuexiao-wang yuexiao-wang deleted the optimize-deploy-example branch August 16, 2016 00:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants