Skip to content
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

Unlist quarkus-resteasy-qute and quarkus-resteasy-reactive-qute #1052

Merged
merged 1 commit into from
Nov 28, 2023

Conversation

ia3andy
Copy link
Contributor

@ia3andy ia3andy commented Nov 27, 2023

part of #37000

@ia3andy
Copy link
Contributor Author

ia3andy commented Nov 27, 2023

cc @mkouba @gsmet

This comment has been minimized.

Copy link
Contributor

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

@aloubyansky
Copy link
Member

Was there a PR unlisting these in core?

@ia3andy
Copy link
Contributor Author

ia3andy commented Nov 28, 2023

@aloubyansky should we unlist them in core? if that's the case we don't need to do it here...

@aloubyansky
Copy link
Member

Yes, they should be unlisted in core unless there is a reason not to.

@ia3andy
Copy link
Contributor Author

ia3andy commented Nov 28, 2023

@aloubyansky ah but they should be also unlisted here (just for this next platform release as it's not yet in core)

@ia3andy ia3andy reopened this Nov 28, 2023
@aloubyansky
Copy link
Member

Sure, no problem.

Copy link

quarkus-bot bot commented Nov 28, 2023

✔️ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

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

OK, let's merge it for now and, ideally, let's drop it when fixed in core.

@gsmet gsmet merged commit 94877d4 into main Nov 28, 2023
9 checks passed
@gsmet gsmet deleted the ia3andy-patch-1 branch December 5, 2023 08:51
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.

4 participants