Skip to content

Conversation

@soltysh
Copy link

@soltysh soltysh commented Jun 8, 2015

This PR waits for #444 being merged first. I'll rebase on top of it.

@danmcp @bparees ptal in the meantime

@soltysh
Copy link
Author

soltysh commented Jun 8, 2015

This fixes #424.

Copy link
Contributor

Choose a reason for hiding this comment

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

we should be able to avoid quoting the whole buildconfig again, since @mfojtik is adding a buildconfig example in this very same file.

@soltysh
Copy link
Author

soltysh commented Jun 9, 2015

Addressed @bparees comments. Fixed formatting in a couple of places. Ready for review and hopefully for merge.
@bparees @adellape ptal.

Copy link
Contributor

Choose a reason for hiding this comment

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

BuildConfig ?

Copy link
Author

Choose a reason for hiding this comment

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

I was thinking of my paragraph as being the continuation of the previous one, where buildConfig is used.

@bparees
Copy link
Contributor

bparees commented Jun 9, 2015

@soltysh 2 nits and lgtm.

@soltysh
Copy link
Author

soltysh commented Jun 9, 2015

2nits addressed, one commented.

@bparees
Copy link
Contributor

bparees commented Jun 9, 2015

looks good to me, @adellape merge at will.

Copy link
Contributor

Choose a reason for hiding this comment

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

<1> should be outside the ".

@soltysh
Copy link
Author

soltysh commented Jun 10, 2015

@adellape addressed your comments.

@adellape
Copy link
Contributor

@soltysh Thanks, LGTM. Will merge after build passes.

adellape added a commit that referenced this pull request Jun 10, 2015
Added incremental buildConfig
@adellape adellape merged commit 1470c09 into openshift:master Jun 10, 2015
@soltysh soltysh deleted the issue424 branch June 10, 2015 21:38
ahardin-rh added a commit that referenced this pull request Dec 15, 2016
sbeskin-redhat pushed a commit to sbeskin-redhat/openshift-docs that referenced this pull request Oct 24, 2022
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