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

Lite/Full split: decide on "Defining new scope types" #494

Closed
Ladicek opened this issue May 20, 2021 · 4 comments
Closed

Lite/Full split: decide on "Defining new scope types" #494

Ladicek opened this issue May 20, 2021 · 4 comments
Assignees
Labels
Lite Related to CDI Lite

Comments

@Ladicek
Copy link
Contributor

Ladicek commented May 20, 2021

The "Defining new scope types" chapter has a TODO about allowing @Priority on stereotypes. This should be decided separately, out of the Lite/Full split. The TODO itself can/should be removed without action.

https://github.com/eclipse-ee4j/cdi/blob/864e1b4f779e4d6d5a80f147d975a510446d4987/spec/src/main/asciidoc/core/definition.asciidoc

@Ladicek
Copy link
Contributor Author

Ladicek commented May 20, 2021

I have filed #495 for deciding whether we want to allow @Priority on stereotypes or not. As I said, the TODO for this (that I added during Lite/Full split) can/should be removed without action, as this decision is unrelated to Lite/Full split.

@Ladicek Ladicek added the Lite Related to CDI Lite label May 24, 2021
@manovotn
Copy link
Contributor

This can probably be closed as #524 will address it but I'll let @Ladicek be the judge of that :)

@Ladicek
Copy link
Contributor Author

Ladicek commented Sep 14, 2021

I'd treat this issue as "we should remove the TODO comment" :-)

@manovotn manovotn self-assigned this Oct 7, 2021
@manovotn
Copy link
Contributor

manovotn commented Oct 7, 2021

Actually, this issue has already been solved in another commit and the TODO note was removed.

@manovotn manovotn closed this as completed Oct 7, 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

No branches or pull requests

2 participants