Skip to content
This repository was archived by the owner on May 13, 2025. It is now read-only.

Comments

[JBEAP-22930] Mojarra -- test branch used for EAP 7.4 using SE 17#99

Open
jasondlee wants to merge 4 commits intowildfly:2.3.14.SPfrom
jasondlee:JBEAP-22930
Open

[JBEAP-22930] Mojarra -- test branch used for EAP 7.4 using SE 17#99
jasondlee wants to merge 4 commits intowildfly:2.3.14.SPfrom
jasondlee:JBEAP-22930

Conversation

@jasondlee
Copy link
Member

https://issues.redhat.com/browse/JBEAP-22930

Modify version in POM to add support for Java 17
Minor test update to work around minor data change in testing Add GH Action workflow

Modify version in POM to add support for Java 17
Minor test update to work around minor data change in testing
Add GH Action workflow
@jasondlee jasondlee marked this pull request as ready for review March 29, 2024 00:14
@jasondlee jasondlee requested review from bstansberry and fjuma April 1, 2024 17:15
@jasondlee
Copy link
Member Author

@bstansberry , @fjuma I would appreciate a quick sanity check if/when one or both of you have a chance. Seems like a safe enough change...

</requireJavaVersion>
</rules>
</configuration>
</plugin>

Choose a reason for hiding this comment

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

If you remove this you need something to enforce the use of SE 8 when releasing. For example:

https://github.com/wildfly/wildfly/blob/main/pom.xml#L1133-L1146

So I think you need to leave the plugin in place but just tweak the execution by moving the configuration into it and setting the phase to 'deploy'.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants