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

Adding force parameter to the bindSingleton method #973

Merged
merged 3 commits into from
Aug 18, 2023

Conversation

msmakouz
Copy link
Member

@msmakouz msmakouz commented Aug 17, 2023

Q A
Bugfix? ✔️
Breaks BC?
New feature? ✔️

What was changed

Added the force parameter to the bindSingleton method with a default value of true. This ensures that the behavior remains unchanged in the minor version. In the major version v4.0.0, the parameter can be removed or set to true by default.

@msmakouz msmakouz added this to the 3.8 milestone Aug 17, 2023
@msmakouz msmakouz self-assigned this Aug 17, 2023
@codecov
Copy link

codecov bot commented Aug 17, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (19d0904) 83.74% compared to head (23b9dae) 83.74%.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #973   +/-   ##
=========================================
  Coverage     83.74%   83.74%           
- Complexity     6040     6041    +1     
=========================================
  Files           793      793           
  Lines         17019    17021    +2     
=========================================
+ Hits          14253    14255    +2     
  Misses         2766     2766           
Files Changed Coverage Δ
src/Core/src/Container.php 94.01% <100.00%> (+0.10%) ⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@msmakouz msmakouz linked an issue Aug 17, 2023 that may be closed by this pull request
src/Core/src/Container.php Outdated Show resolved Hide resolved
@butschster butschster merged commit 749d314 into spiral:master Aug 18, 2023
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[spiral/core] Add force binding singletons
3 participants