Update supported Helidon releases; add SE 17 info along the way#481
Update supported Helidon releases; add SE 17 info along the way#481tjquinno wants to merge 1 commit intomicroprofile:mainfrom
Conversation
|
@Karm When you can, please take a look. Thanks in advance. |
|
Looking into the test failures. |
|
@Karm It looks as if my PR does not completely enable the use of Java 17 where needed. My local testing fails when Maven rejects 17 as a choice. I know @Emily-Jiang mentioned that support for 17 would be added. Maybe I should let the professionals take care of that in a separate PR and then I can present this one again. Or, if you want to coach me as to what other changes I should add to this one, I'd be happy to do that too. Whoever is easier for you. |
|
@tjquinno can you separate your PR to 2 parts? One PR is to update Helidon while the other one is to add Java 17. |
|
Hi, @Emily-Jiang. As I noted in my earlier comment, these changes are not sufficient for adding Java 17 support. Someone more knowledgeable than I will need to do that part. Then I am happy to modify this PR to address just the Helidon-specific changes. |
thanks @tjquinno ! Just rework your PR to add Helidon and we will add Java 17 under a different PR. @Karm Would you like to take a look at adding Java 17? |
Resolves #478