Skip to content

Conversation

@bzssm
Copy link

@bzssm bzssm commented Jan 9, 2026

Description of Changes

add java jakarta ee framework detection support

Related Issue(s)

Link the GitHub/GitLab/JIRA issues that are related to this PR.

Acceptance Criteria

Testing and documentation do not need to be complete in order for this PR to be approved. However, tracking issues must be opened for missing testing/documentation.

  • Unit/Functional tests

  • Documentation

Tests Performed

Explain what tests you personally ran to ensure the changes are functioning as expected.

How To Test

Instructions for the reviewer on how to test your changes.

Notes To Reviewer

Any notes you would like to include for the reviewer.

@bzssm
Copy link
Author

bzssm commented Jan 9, 2026

Hi @thepetk @Jdubrick @elsony @michael-valdron kindly reqeust your review on this PR. Thanks!

@bzssm bzssm force-pushed the royyang/add-jakartaee-support branch from a8c0f8c to 7fb4963 Compare January 9, 2026 08:03
Copy link
Contributor

@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.

nice work @bzssm !! Thank you for your contribution

Before I approve, I think is important to have some doc updates too! Thanks!

@openshift-ci
Copy link

openshift-ci bot commented Jan 12, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: bzssm
Once this PR has been reviewed and has the lgtm label, please ask for approval from thepetk. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@bzssm
Copy link
Author

bzssm commented Jan 12, 2026

Hi @thepetk , thanks for the feedback!
I've added the documentation update in the latest commit - JakartaEE is now listed in the supported Java frameworks section of docs/public/alizer-spec.md.

Please take another look when you have a chance. Thanks!

@thepetk
Copy link
Contributor

thepetk commented Jan 12, 2026

@bzssm looks good in general, it only needs a rebase after we have merged another PR. Thanks!

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