Skip to content

Decide on other mentions of Full-only concepts in the CDI Lite spec p…#541

Merged
Ladicek merged 1 commit intojakartaee:masterfrom
manovotn:issue489
Oct 12, 2021
Merged

Decide on other mentions of Full-only concepts in the CDI Lite spec p…#541
Ladicek merged 1 commit intojakartaee:masterfrom
manovotn:issue489

Conversation

@manovotn
Copy link
Copy Markdown
Contributor

@manovotn manovotn commented Oct 11, 2021

…art.

Fixes #489

This PR moves some more CDI Full mentions into their respective specification sections.

@manovotn manovotn requested a review from Ladicek October 11, 2021 14:18
@manovotn manovotn force-pushed the issue489 branch 3 times, most recently from c493a2f to 2219e21 Compare October 11, 2021 14:56
@manovotn
Copy link
Copy Markdown
Contributor Author

@Ladicek thanks for comments, all of them should be addressed now.

@manovotn manovotn requested a review from Ladicek October 11, 2021 14:57
@manovotn
Copy link
Copy Markdown
Contributor Author

@Ladicek are you fine with the state of this PR?

@Ladicek
Copy link
Copy Markdown
Member

Ladicek commented Oct 12, 2021

Sorry it took me so long. These are the final comments I have.

@manovotn
Copy link
Copy Markdown
Contributor Author

@Ladicek addressed both comments

@Ladicek Ladicek merged commit 269ef0d into jakartaee:master Oct 12, 2021
@Ladicek
Copy link
Copy Markdown
Member

Ladicek commented Oct 12, 2021

Thanks!

@manovotn manovotn deleted the issue489 branch October 12, 2021 14:26
@Ladicek Ladicek added the Lite Related to CDI Lite label Oct 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Lite Related to CDI Lite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Lite/Full split: decide on other mentions of Full-only concepts in the CDI Lite spec part

2 participants