-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[FrameworkBundle] Add semaphore configuration #44155
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
Conversation
1be744d
to
6ff6663
Compare
6ff6663
to
9abc6d1
Compare
@jderusse Is it ready to be merged? Is yes, can you rebase? |
src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(with one question)
@jderusse Last chance for 6.1 :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks.
Let's fix the code for lock on 4.4
Thank you @jderusse. |
…on (jderusse) This PR was merged into the 4.4 branch. Discussion ---------- [FrameworkBundle] Fix env resolution in lock configuration | Q | A | ------------- | --- | Branch? | 4.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | / | License | MIT | Doc PR | / Fix wrong value injected in lock factory. See #44155 (comment) Commits ------- c1a9af7 Fix env resolution in lock configuration
…on (jderusse) This PR was merged into the 4.4 branch. Discussion ---------- [FrameworkBundle] Fix env resolution in lock configuration | Q | A | ------------- | --- | Branch? | 4.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | / | License | MIT | Doc PR | / Fix wrong value injected in lock factory. See symfony/symfony#44155 (comment) Commits ------- c1a9af7a0b Fix env resolution in lock configuration
Adds a new configuration for semaphores