Skip to content

add Lang Model TCK coverage for sealed classes#623

Merged
manovotn merged 4 commits intojakartaee:masterfrom
Ladicek:lang-model-sealed-classes
Mar 20, 2025
Merged

add Lang Model TCK coverage for sealed classes#623
manovotn merged 4 commits intojakartaee:masterfrom
Ladicek:lang-model-sealed-classes

Conversation

@Ladicek
Copy link
Copy Markdown
Member

@Ladicek Ladicek commented Feb 27, 2025

TCK coverage for jakartaee/cdi#857

@Ladicek Ladicek added this to the CDI 5.0 milestone Feb 27, 2025
@Ladicek
Copy link
Copy Markdown
Member Author

Ladicek commented Feb 27, 2025

CI is supposed to fail ATM, because of a SNAPSHOT dependency.

@Ladicek Ladicek requested review from Azquelt and manovotn February 27, 2025 13:47
@manovotn
Copy link
Copy Markdown
Contributor

manovotn commented Mar 8, 2025

CI is supposed to fail ATM, because of a SNAPSHOT dependency.

@Ladicek @Azquelt unless you have something against it, I'd propose we start doing Alpha releases (for both, CDI and TCK) like we did in previous versions. That will allow us to merge TCK PRs and impls can also consume it to keep up to date with spec changes.

@Ladicek
Copy link
Copy Markdown
Member Author

Ladicek commented Mar 10, 2025

Agree.

@Azquelt
Copy link
Copy Markdown
Member

Azquelt commented Mar 10, 2025

I agree as well

@manovotn
Copy link
Copy Markdown
Contributor

manovotn commented Mar 20, 2025

After sorting out some issues with SSH key in Jenkins, the CDI API Alpha1 is now in Central.
I've added a commit updating it here - the CI should now turn green.

@manovotn manovotn merged commit 1c3aacc into jakartaee:master Mar 20, 2025
3 checks passed
@Ladicek Ladicek deleted the lang-model-sealed-classes branch March 24, 2025 07:59
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.

3 participants