Skip to content

Polish BeanRegistrar Javadoc and add tests for non-invocation semantics#36581

Merged
jhoeller merged 1 commit intospring-projects:mainfrom
snicoll:bean-registrar-polish
Apr 2, 2026
Merged

Polish BeanRegistrar Javadoc and add tests for non-invocation semantics#36581
jhoeller merged 1 commit intospring-projects:mainfrom
snicoll:bean-registrar-polish

Conversation

@snicoll
Copy link
Copy Markdown
Member

@snicoll snicoll commented Apr 2, 2026

Revise the BeanRegistrar Javadoc to document the two distinct usage modes: @Configuration/@import and programmatic GenericApplicationContext setup.

Clarify that implementations are not Spring components (requiring a no-arg constructor and no dependency injection), and detail the ordering guarantees for each mode.

Add missing tests

Revise the BeanRegistrar Javadoc to document the two distinct usage
modes: @Configuration/@import and programmatic GenericApplicationContext
setup.

Clarify that implementations are not Spring components (requiring a
no-arg constructor and no dependency injection), and detail the ordering
guarantees for each mode.

Add missing tests

Signed-off-by: Stéphane Nicoll <stephane.nicoll@broadcom.com>
@snicoll snicoll force-pushed the bean-registrar-polish branch from 43e5671 to 0f761df Compare April 2, 2026 09:00
@jhoeller jhoeller self-assigned this Apr 2, 2026
@jhoeller jhoeller added type: task A general task in: core Issues in core modules (aop, beans, core, context, expression) labels Apr 2, 2026
@jhoeller jhoeller added this to the 7.1.0-M1 milestone Apr 2, 2026
@jhoeller jhoeller merged commit d5b6f4a into spring-projects:main Apr 2, 2026
4 checks passed
sbrannen added a commit that referenced this pull request Apr 2, 2026
snicoll added a commit that referenced this pull request Apr 2, 2026
@snicoll snicoll deleted the bean-registrar-polish branch April 2, 2026 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in: core Issues in core modules (aop, beans, core, context, expression) type: task A general task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants