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

introduce javaMajor variable in template (#160) #187

Merged
merged 1 commit into from
Nov 14, 2018

Conversation

vorburger
Copy link
Collaborator

No description provided.

@vorburger vorburger requested a review from rhuss November 7, 2018 23:04
@vorburger
Copy link
Collaborator Author

@rhuss this will let us switch 1.8.0 vs 11 ... OK for you?

@vorburger
Copy link
Collaborator Author

NB This extends #182, when reviewing this PR, just look at the 2nd commit 5104a0d.

@vorburger
Copy link
Collaborator Author

@rhuss and any other active maintainer reading this, do you have any objections to merging this one?

@rhuss
Copy link
Contributor

rhuss commented Nov 13, 2018

Sorry, I'm just at a f2f meeting with little to no spare time. So feel free to merge yourself, we can discuss this later still, too.

@vorburger
Copy link
Collaborator Author

TODO Following #182, I'll remove that commit from this PR when I rebase it.

@vorburger
Copy link
Collaborator Author

TODO Following #182, I'll remove that commit from this PR when I rebase it.

Done; @rhuss and other commiters, willing to review and merge this one?

@rhuss
Copy link
Contributor

rhuss commented Nov 13, 2018

I'm afraid, there's no one left ;-) Gimme that evening, hopefully I can jump on it. If not, feel free to merge and I will do a review in retrospective.

@vorburger vorburger merged commit 1752c44 into fabric8io-images:master Nov 14, 2018
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