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

DeploymentDescriptorFull/WebTests Adds Resource Definitions with Qualifiers on a CDI Bean #494

Merged
merged 1 commit into from
May 14, 2024

Conversation

aubi
Copy link
Contributor

@aubi aubi commented May 14, 2024

Adding the same annotation as in #491

DeploymentDescriptorFull/WebTests are basically the same tests as AnnotationFull/WebTests, just requiring loading the definitions from xml files.

@aubi aubi requested review from njr-11 and arjantijms May 14, 2024 09:15
@njr-11
Copy link
Contributor

njr-11 commented May 14, 2024

The change looks fine and is nothing objectionable, but to confirm, do we need to hold up the 3.1 release for this? It will be the second time now that we have gone and prepared published TCK results for the component specification release review only to find that we need to redo everything all over again because of an update to the TCK, which also delays getting the spec to ballot.

@njr-11 njr-11 added the TCK label May 14, 2024
@KyleAure KyleAure merged commit 262685a into jakartaee:main May 14, 2024
3 checks passed
@KyleAure KyleAure mentioned this pull request May 14, 2024
22 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants