Skip to content

Scan EJB Interface annotations of RESTFul endpoints #33694

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
Open
WhiteCat22 wants to merge 5 commits into
base: integration
Choose a base branch
from
Open

Scan EJB Interface annotations of RESTFul endpoints #33694

WhiteCat22 wants to merge 5 commits into from

Conversation

@WhiteCat22
Copy link
Member

@WhiteCat22 WhiteCat22 commented Dec 16, 2025 •
edited
Loading

Fixes #31493

@WhiteCat22 WhiteCat22 self-assigned this Dec 16, 2025
@WhiteCat22 WhiteCat22 force-pushed the restful_ejb2 branch 3 times, most recently from b788b78 to a4dfc47 Compare December 16, 2025 23:42
@WhiteCat22
Copy link
Member Author

WhiteCat22 commented Dec 16, 2025 •
edited by LibbyBot
Loading

#libby #build #spawn.fullfat.buckets=com.ibm.ws.jaxrs.2.0_fat (view Open Liberty Personal Build - ✅ completed successfully!)

Note: Target locations of links might be accessible only to IBM employees.

jamezp
jamezp approved these changes Dec 16, 2025
View reviewed changes
Copy link

@jamezp jamezp left a comment

Choose a reason for hiding this comment

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

Looks good to me. Just a minor comment that doesn't need to be addressed.

@LibbyBot
Copy link
Member

LibbyBot commented Dec 16, 2025

Code analysis and actions

DO NOT DELETE THIS COMMENT.

  • 20 FAT files were changed, added, or removed.

  • Check that the build did not break the affected FAT suite(s).

  • 7 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

@WhiteCat22 WhiteCat22 mentioned this pull request Dec 17, 2025
Closed
3 tasks
jhanders34
jhanders34 requested changes Dec 17, 2025
View reviewed changes
@WhiteCat22
Copy link
Member Author

WhiteCat22 commented Dec 19, 2025 •
edited by LibbyBot
Loading

#libby #build #spawn.fullfat.buckets=com.ibm.ws.jaxrs.2.0_fat (view Open Liberty Personal Build - ❌ completed with errors/failures.)

Note: Target locations of links might be accessible only to IBM employees.

@LibbyBot
Copy link
Member

LibbyBot commented Dec 19, 2025

Code analysis and actions

DO NOT DELETE THIS COMMENT.

  • 20 FAT files were changed, added, or removed.

  • Check that the build did not break the affected FAT suite(s).

  • 7 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

@WhiteCat22 WhiteCat22 force-pushed the restful_ejb2 branch 2 times, most recently from cd32b42 to 2c9208e Compare December 19, 2025 20:07
@WhiteCat22
Copy link
Member Author

WhiteCat22 commented Dec 19, 2025

force pushed to sign my commits, no code changes from previous build

@WhiteCat22
Copy link
Member Author

WhiteCat22 commented Dec 19, 2025

#libby

@WhiteCat22
Copy link
Member Author

WhiteCat22 commented Dec 29, 2025 •
edited by LibbyBot
Loading

#libby #build #spawn.fullfat.buckets=com.ibm.ws.jaxrs.2.0_fat
(view Open Liberty Personal Build - ⏳InProgress)

Need to run the copyright header check after signing the commits. Requires a full build and not just #libby I guess.

Note: Target locations of links might be accessible only to IBM employees.

@LibbyBot
Copy link
Member

LibbyBot commented Dec 29, 2025

Code analysis and actions

DO NOT DELETE THIS COMMENT.

  • 20 FAT files were changed, added, or removed.

  • Check that the build did not break the affected FAT suite(s).

  • 7 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

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

Reviewers

@jhanders34 jhanders34 jhanders34 approved these changes

@jim-krueger jim-krueger jim-krueger approved these changes

+1 more reviewer

@jamezp jamezp jamezp approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@WhiteCat22 WhiteCat22

Labels

in:JAX-RS

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

SessionContext is null

5 participants

@WhiteCat22 @LibbyBot @jamezp @jhanders34 @jim-krueger