Skip to content

Improve support for Jakarta EE 11 #8229

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
Merged
pepness merged 1 commit into from
Feb 17, 2025
Merged

Improve support for Jakarta EE 11 #8229

pepness merged 1 commit into from
Feb 17, 2025

Conversation

@pepness
Copy link
Member

@pepness pepness commented Feb 11, 2025

NetBeans Notes:

  • Jakarta EE 11 runs on Java SE 17 and later
  • Add missing break to switch statement
  • Assume Jakarta EE 8 is supported instead of Java EE 7 when platformVersion is null when dealing with EJBs

NetBeans Testing:

  • Verify successful execution of libraries and licenses Ant test
  • Verify successful execution of Verify Sigtests
  • Verify successful execution of unit tests for modules j2ee.ejbjarproject, javaee.project, and tomcat5
  • Started NetBeans and ensure the log didn't have any ERROR or new WARNINGS
  • Successfully register TomEE 10 and GlassFish 8.0.0-M9
    • Create a Jakarta EE 11 web app and verify that it works
    • Create a Jakarta EE 10 web app and verify that it works.

@pepness
-Assume Jakarta EE 8 is supported when platformVersion is null when
1b9bea6 
 dealing with EJB.
-Jakarta EE 11 runs on Java SE 17 and later
-Add missing 'break' to switch
@pepness pepness added Java EE/Jakarta EE [ci] enable enterprise job enterprise [ci] enable enterprise job labels Feb 11, 2025
@pepness pepness added this to the NB26 milestone Feb 11, 2025
@pepness pepness self-assigned this Feb 11, 2025
matthiasblaesing
matthiasblaesing approved these changes Feb 16, 2025
View reviewed changes
Copy link
Contributor

@matthiasblaesing matthiasblaesing left a comment

Choose a reason for hiding this comment

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

Eyeballed and looks sane to me.

@pepness pepness merged commit ee71e88 into apache:master Feb 17, 2025
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@matthiasblaesing matthiasblaesing matthiasblaesing approved these changes

Assignees

@pepness pepness

Labels

enterprise [ci] enable enterprise job Java EE/Jakarta EE [ci] enable enterprise job

Projects

None yet

Milestone

NB26

Development

Successfully merging this pull request may close these issues.

2 participants

@pepness @matthiasblaesing