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

Revert support for scanning resource methods without JAX-RS HTTP methods #1238

Merged
merged 1 commit into from
Sep 5, 2022
Merged

Revert support for scanning resource methods without JAX-RS HTTP methods #1238

merged 1 commit into from
Sep 5, 2022

Conversation

MikeEdgar
Copy link
Member

Fixes #1234

@MikeEdgar MikeEdgar added the backport/2.x Issue pending backport to the 2.x series label Sep 2, 2022
@MikeEdgar MikeEdgar added this to the 3.1.0 milestone Sep 2, 2022
@MikeEdgar MikeEdgar removed the backport/2.x Issue pending backport to the 2.x series label Sep 2, 2022
@MikeEdgar MikeEdgar force-pushed the issue-1234 branch 2 times, most recently from 7d5b970 to d691978 Compare September 2, 2022 23:34
snazy
snazy previously approved these changes Sep 5, 2022
View reviewed changes
Copy link
Contributor

@snazy snazy left a comment

Choose a reason for hiding this comment

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

Tried this commit in projectnessie - confirm #1234 is fixed w/ this change

@MikeEdgar
Copy link
Member Author

@phillip-kruger - this is the same as #1239. I'll merge after the CI passes.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 5, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@MikeEdgar MikeEdgar merged commit ba62b01 into smallrye:main Sep 5, 2022
@MikeEdgar MikeEdgar deleted the issue-1234 branch September 5, 2022 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@snazy snazy snazy left review comments

@phillip-kruger phillip-kruger Awaiting requested review from phillip-kruger

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
3.1.0
Development

Successfully merging this pull request may close these issues.

Wrong response type (current HEAD of main)
2 participants
@MikeEdgar @snazy