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.

Sign up for GitHub

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

Jump to bottom

Bump Java version to let CI succeed #105

Closed
ckittl opened this issue Feb 14, 2022 · 2 comments · Fixed by #125
Closed

Bump Java version to let CI succeed #105

ckittl opened this issue Feb 14, 2022 · 2 comments · Fixed by #125
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@ckittl
Copy link
Member

ckittl commented Feb 14, 2022

Bump Java version to sth. >= 11

Originally posted by @t-ober in #96 (comment)
@ckittl ckittl added bug Something isn't working good first issue Good for newcomers labels Feb 14, 2022
@RishiKumarRay
Copy link

@ckittl maybe i am wrong but if CI is being built with jenkins the changing def javaVersionId = 'jdk-8' in jenkins file will fix this issue?

@sebastian-peter sebastian-peter mentioned this issue Apr 1, 2022
Merged
t-ober added a commit that referenced this issue Apr 1, 2022
@t-ober
Merge pull request #125 from ie3-institute/sp/#105-upgrade-to-jdk17
5e11244 
Upgrade to jdk17
@sebastian-peter
Copy link
Member

sebastian-peter commented Apr 1, 2022 •
edited
Loading

maybe i am wrong but if CI is being built with jenkins the changing def javaVersionId = 'jdk-8' in jenkins file will fix this issue?

Yes, among some other things. You can check out the PR #125 if you're interested

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Upgrade to jdk17 ie3-institute/DWDWeatherTools
3 participants
@sebastian-peter @ckittl @RishiKumarRay