Skip to content

Pin the parent Devfile version to 1.2.0 #36

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

Merged
merged 2 commits into from
Oct 20, 2023

Conversation

rm3l
Copy link
Contributor

@rm3l rm3l commented Oct 19, 2023

The default version of the parent Stack will be updated in [1] to use JDK 17,
so that users on recent versions of Spring Boot can use such Stack.

This PR is to make sure this sample does not break with such a change,
and continues to work as before, since it is still using Sprint Boot 2.x.

[1] devfile/registry#169

rm3l added 2 commits October 19, 2023 13:50
The default version of the parent Stack will be updated in [1] to use JDK 17,
so that users on recent versions of Spring Boot can use such Stack.

This is to make sure this sample does not break with such change,
and continues to work as before, since it is using Sprint Boot 2.x (and
expects JDK 11).

Signed-off-by: Armel Soro <[email protected]>
Recent versions of odo add additional files to this folder.

Signed-off-by: Armel Soro <[email protected]>
@rm3l
Copy link
Contributor Author

rm3l commented Oct 19, 2023

@thepetk PTAL at this PR following our discussion. I've tested this with odo and ODC and did not run into any issues.

Copy link

@thepetk thepetk left a comment

Choose a reason for hiding this comment

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

/lgtm

@thepetk thepetk merged commit 2f8d1e8 into devfile-samples:main Oct 20, 2023
@rm3l rm3l deleted the pin_parent_devfile_to_1.2.0 branch October 20, 2023 13:20
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