Skip to content

[main-lts] Generate Builder methods on Beans #1230

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.

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

Open
wants to merge 4 commits into
base: main-lts
Choose a base branch
from

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jul 4, 2025

Backport: #1162

Note: CI is not automatically triggered on backported PRs, please comment 'ok to test' to launch Jenkins jobs

Implement #1160

Expose the option generateBuilders from jsonschema2pojo

@github-actions github-actions bot requested a review from a team as a code owner July 4, 2025 13:36
@mcruzdev mcruzdev closed this Jul 5, 2025
@mcruzdev mcruzdev reopened this Jul 5, 2025
@hbelmiro
Copy link
Contributor

hbelmiro commented Jul 7, 2025

@mcruzdev the CI is failing. Can you PTAL?

@mcruzdev
Copy link
Member

mcruzdev commented Jul 7, 2025

@hbelmiro I think we had a problem with an old cherry pick related to codestarts. This pull request adds the codestarts module into server/integrations-tests, but there is no codestarts module there.

@ricardozanini
Copy link
Member

@mcruzdev please open a new PR on main-lts cherry picking manually. The automation failed - has happened before.

@crazybolillo
Copy link

Can I help in any way? Looking forward to having this merged so I can use the builder pattern, thanks.

@mcruzdev
Copy link
Member

Hi @ricardozanini, sorry for delay! I am tracking this issue, I am waiting a new version from Apicurio codegen. Having this one I will do the cherry pick.

Copy link
Contributor Author

🎊 PR Preview 6bc5faa has been successfully built and deployed. See the documentation preview: https://quarkus-openapi-generator-preview-pr-1230.surge.sh

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

Successfully merging this pull request may close these issues.

5 participants