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

Push image build in from PR #136

Closed

Conversation

vpavlin
Copy link
Member

@vpavlin vpavlin commented Feb 9, 2018

No description provided.

@vpavlin vpavlin force-pushed the feature/push-PR-build branch 3 times, most recently from fceb724 to e8c0fda Compare February 9, 2018 14:03
@vpavlin vpavlin force-pushed the feature/push-PR-build branch from e8c0fda to 7f149ec Compare February 9, 2018 14:07
@vpavlin vpavlin requested a review from hferentschik February 9, 2018 14:10
@hferentschik
Copy link
Contributor

Do we still want to merge this? What is the idea? Each pull request builds an image and pushes it? Then you can manually deploy it?

@vpavlin
Copy link
Member Author

vpavlin commented Feb 12, 2018

Yeah, that was my idea to try out, but with your solution to run locally, we might not need to pollute the registry

@hferentschik
Copy link
Contributor

Yeah, that was my idea to try out, but with your solution to run locally, we might not need to pollute the registry

I actually like this, but we can see how far we get with the ability to run locally. I created issue #147 to track this and we can always dig out this change of yours this way. Closing this pull request for now.

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.

2 participants