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

CDI-664 Rename/clarify purpose of few SeContainerInitializere methods #360

Merged
merged 1 commit into from
Dec 12, 2016

Conversation

mkouba
Copy link
Contributor

@mkouba mkouba commented Dec 8, 2016

No description provided.

* `enableInterceptors()` adds interceptoror classes to the list of enabled interceptors for the synthetic bean archive.
* `enableDecorators()` adds decorator classes to the list of enabled decorators for the synthetic bean archive.
* `selectAlternatives()` adds alternatives classes to the list of selected alternatives for the synthetic bean archive.
* `selectAlternativeStereotypes()` alternative stereotype classes to the list of selected alternative stereotypes for the synthetic bean archive.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing adds?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch again!! ;-)


/**
* Add decorators for the synthetic bean archive. Decorator classes are automatically added to the set of bean classes for the synthetic bean archive.
* Add decorator classes to the list of enabled decorators for the synthetic bean archive.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would add This method does not add any class to the set of bean classes of the synthetic bean archive. here as well

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense. Will add this to "alternatives" methods as well.

@manovotn
Copy link
Contributor

manovotn commented Dec 9, 2016

I think it's ok now

Copy link
Contributor

@antoinesd antoinesd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@antoinesd antoinesd merged commit 7d6d8d2 into jakartaee:master Dec 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants