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.

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

Properly handle void return type in REST Client Reactive #23472

Merged
merged 1 commit into from
Feb 7, 2022
Merged

Properly handle void return type in REST Client Reactive #23472

merged 1 commit into from
Feb 7, 2022

Conversation

geoand
Copy link
Contributor

@geoand geoand commented Feb 7, 2022

Fixes: #22830

@quarkus-bot quarkus-bot bot added the area/rest label Feb 7, 2022
@michalszynkiewicz
Copy link
Member

@geoand do you know if we play well with Uni<Void>, etc?

@geoand
Copy link
Contributor Author

geoand commented Feb 7, 2022

Yup, I checked it and it works as expected

@geoand geoand added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Feb 7, 2022
@michalszynkiewicz
Copy link
Member

Yup, I checked it and it works as expected

thanks

@geoand geoand merged commit c19f956 into quarkusio:main Feb 7, 2022
@quarkus-bot quarkus-bot bot added this to the 2.8 - main milestone Feb 7, 2022
@geoand geoand deleted the #22830 branch February 7, 2022 12:06
@quarkus-bot quarkus-bot bot added kind/bugfix and removed triage/waiting-for-ci Ready to merge when CI successfully finishes labels Feb 7, 2022
@gsmet gsmet modified the milestones: 2.8 - main, 2.7.1.Final Feb 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@michalszynkiewicz michalszynkiewicz michalszynkiewicz approved these changes

Assignees
No one assigned
Labels
area/rest kind/bugfix
Projects
None yet
Milestone
2.7.1.Final
Development

Successfully merging this pull request may close these issues.

Rest-Client-Reactive: void methods not allowed
3 participants
@geoand @michalszynkiewicz @gsmet