Skip to content

Conversation

@haoozhang
Copy link
Contributor

@haoozhang haoozhang commented Dec 3, 2025

Description of Changes

This PR is to improve the detection of JBoss EAP framework.

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.

[x] For each added clue, add the test project and unit test to verify it.

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.

References

  1. https://docs.redhat.com/en/documentation/red_hat_jboss_enterprise_application_platform/7.3/html/development_guide/using_maven_with_eap
  2. https://docs.redhat.com/en/documentation/red_hat_jboss_enterprise_application_platform/7.2/html/development_guide/using_maven_with_eap
  3. https://docs.redhat.com/en/documentation/red_hat_jboss_enterprise_application_platform/7.0/html/development_guide/using_maven_with_eap
  4. https://docs.redhat.com/en/documentation/red_hat_jboss_enterprise_application_platform/6.4/html/development_guide/sect-use_the_maven_repository#Configure_the_JBoss_Enterprise_Application_Platform_Maven_Repository_Using_the_Project_POM

@openshift-ci
Copy link

openshift-ci bot commented Dec 3, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: haoozhang
Once this PR has been reviewed and has the lgtm label, please assign michael-valdron for approval. 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

@haoozhang haoozhang marked this pull request as ready for review December 9, 2025 10:06
@haoozhang
Copy link
Contributor Author

haoozhang commented Dec 9, 2025

Hi @elsony @michael-valdron @Jdubrick @thepetk , could you please help to review this PR, thanks!

HaoZhang added 2 commits December 9, 2025 19:51
Signed-off-by: HaoZhang <haozhan@microsoft.com>
Signed-off-by: HaoZhang <haozhan@microsoft.com>

# Conflicts:
#	test/apis/component_recognizer_test.go
Signed-off-by: HaoZhang <haozhan@microsoft.com>
@thepetk
Copy link
Contributor

thepetk commented Dec 11, 2025

Hey @haoozhang! Thank you for your PR! I'll take a look in the next days :)

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