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.

Sign up for GitHub

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

Jump to bottom

Harmonize handling of current factory method in SimpleInstantiationStrategy #33192

Closed
snicoll opened this issue Jul 10, 2024 · 0 comments
Closed

Harmonize handling of current factory method in SimpleInstantiationStrategy #33192

snicoll opened this issue Jul 10, 2024 · 0 comments
Assignees
@snicoll
Labels
type: enhancement A general enhancement
Milestone
6.2.0-M5

Comments

@snicoll
Copy link
Member

snicoll commented Jul 10, 2024

Working on #33180 revealed that exposing the currently invoked factory method could lead to side effect if not handled properly. Rather than allowing a way to set it, we'd like to make sure all use go through the same template method.
@snicoll snicoll added the type: enhancement A general enhancement label Jul 10, 2024
@snicoll snicoll added this to the 6.2.0-M5 milestone Jul 10, 2024
@snicoll snicoll self-assigned this Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@snicoll snicoll

Labels
type: enhancement A general enhancement
Projects
None yet
Milestone
6.2.0-M5
Development

No branches or pull requests
1 participant
@snicoll