-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
[DependencyInjection] Update debug.rst #15820
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
Did you experience some issue because of these quotes? If I remember correctly, we added these quotes (here and in other commands across the docs) because not adding them causes problems for Windows users. |
Yes, when I added these quotes I got an error message in console and when I
executed without quotes the error was gone. By the way, my develop
environment is win 10.
…On Fri, Sep 17, 2021, 8:15 PM Javier Eguiluz ***@***.***> wrote:
Did you experience some issue because of these quotes?
If I remember correctly, we added these quotes (here and in other commands
across the docs) because not adding them causes problems for Windows users.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#15820 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADTOAYDH4N2ZSJ2NB2JH7BLUCNPAXANCNFSM5EBX5JJA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
Thanks for the feedback! This is now more confusing to me. I remember that when we add unneeded quotes in some command it's because someone using Windows asked us to do that because otherwise it fails. |
You are welcome. By the way, I use without quotes it seems when I add the
quotes the framework thinks it is a part of my service name.
I attached an example for you to this email.
…On Wed, 22 Sept 2021 at 16:40, Javier Eguiluz ***@***.***> wrote:
Thanks for the feedback! This is now more confusing to me. I remember that
when we add unneeded quotes in some command it's because someone using
Windows asked us to do that because otherwise it fails.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#15820 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADTOAYHZ2FFYM3OMDP6ILXTUDHIVVANCNFSM5EBX5JJA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
I've asked in https://symfony.com/slack and many developers using Windows replied. The summary is: if you use the basic So, we're merging this pull request because it makes it work for everybody. Thanks! |
The service ID should not be surrounded by a single quote.
e9f00c2
to
0483ed6
Compare
Thank you Ali ... and congrats on your first Symfony Docs contribution 🎉 |
Thanks for your respond. Good luck
…On Wed, Sep 22, 2021, 5:19 PM Javier Eguiluz ***@***.***> wrote:
I've asked in https://symfony.com/slack and many developers using Windows
replied. The summary is: if you use the basic cmd console, only the
command WITHOUT quotes works; if you use any other advanced
console/terminal, it works WITH and WITHOUT quotes.
So, we're merging this pull request because it makes it work for
everybody. Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#15820 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADTOAYEEXKFET5QDXFZDNDLUDHNFHANCNFSM5EBX5JJA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
|
The service ID should not be surrounded by a single quote.