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

Change InterceptorBindingInheritanceTest to avoid using recursive interception #469

Merged
merged 1 commit into from
May 29, 2023
Merged

Change InterceptorBindingInheritanceTest to avoid using recursive interception #469

merged 1 commit into from
May 29, 2023

Conversation

Ladicek
Copy link
Contributor

@Ladicek Ladicek commented May 29, 2023

Fixes #468

@Ladicek
Copy link
Contributor Author

Ladicek commented May 29, 2023

Draft until the challenge is accepted.

@Ladicek Ladicek marked this pull request as ready for review May 29, 2023 09:18
@manovotn
Copy link
Contributor

I'm going to fast forward this into main as it has both, the labels and the approval and I am about to cut a TCK release (as I promised to do during last CDI call we had).
I'm happy to have further discussion around #468 and #464 if there is need for it.

@manovotn manovotn merged commit a0d35fd into jakartaee:master May 29, 2023
@Ladicek Ladicek deleted the recursive-interception branch May 29, 2023 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@manovotn manovotn manovotn approved these changes

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

Successfully merging this pull request may close these issues.

InterceptorBindingInheritanceTest relies on undefined behavior
2 participants
@Ladicek @manovotn