Skip to content

Convert applicable methods in Scope to default methods - #34757

Merged
jhoeller merged 1 commit into
spring-projects:mainfrom
quaff:patch-117
Jul 7, 2025
Merged

jhoeller merged 1 commit into
spring-projects:mainfrom
quaff:patch-117

Conversation

@quaff

@quaff quaff commented Apr 15, 2025 •

Copy link
Copy Markdown
Contributor

getConversationId and resolveContextualObject are candidates for being converted to default methods, not sure about registerDestructionCallback.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Apr 15, 2025
@sbrannen sbrannen added the in: core Issues in core modules (aop, beans, core, context, expression) label Apr 15, 2025
@sbrannen sbrannen changed the title Mark applicable Scope's methods as default Convert applicable methods in Scope to default methods Apr 15, 2025
@sbrannen
sbrannen requested a review from jhoeller April 15, 2025 09:21
@jhoeller

jhoeller commented Jul 7, 2025 •

Copy link
Copy Markdown
Contributor

@quaff this seems sensible. Please try to rebase it onto current main since there seem to be some conflicts in the meantime.

@jhoeller jhoeller added type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Jul 7, 2025
@jhoeller jhoeller self-assigned this Jul 7, 2025
@jhoeller jhoeller added this to the 7.0.0-M7 milestone Jul 7, 2025
Signed-off-by: Yanming Zhou <zhouyanming@gmail.com>
@quaff

quaff commented Jul 7, 2025

Copy link
Copy Markdown
Contributor Author

@quaff this seems sensible. Olease try to rebase it onto current main since there seem to be some conflicts in the meantime.

Done.

@jhoeller
jhoeller merged commit 410a9ce into spring-projects:main Jul 7, 2025
jhoeller added a commit that referenced this pull request Jul 7, 2025
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: enhancement A general enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants