-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[FrameworkBundle] Add phpstorm ide #20019
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
@@ -100,6 +100,7 @@ public function load(array $configs, ContainerBuilder $container) | |||
'macvim' => 'mvim://open?url=file://%%f&line=%%l', | |||
'emacs' => 'emacs://open?url=file://%%f&line=%%l', | |||
'sublime' => 'subl://open?url=file://%%f&line=%%l', | |||
'phpstorm' => 'pstorm://open/?url=file://%%f&line=%%l', |
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.
Is the slash before "?" necessary?
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.
I created PR aik099/PhpStormProtocol#19
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.
Fixed in aik099/PhpStormProtocol#20
Thank you @hason. |
This PR was merged into the 3.2-dev branch. Discussion ---------- [FrameworkBundle] Add phpstorm ide | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | yes | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | License | MIT | Doc PR | Works on Windows, Mac (https://github.com/aik099/PhpStormProtocol) and Linux (sanduhrs/phpstorm-url-handler@8f7a8df). Commits ------- 450c37c [FrameworkBundle] Add phpstorm ide
This PR was merged into the master branch. Discussion ---------- Updated the framework.ide description - PHPStorm was added here but it was never documented: symfony/symfony#20019 - PhpStormProtocol is not only needed for Windows: https://github.com/aik099/PhpStormProtocol Commits ------- b619fc2 Updated the framework.ide description
Works on Windows, Mac (https://github.com/aik099/PhpStormProtocol) and Linux (sanduhrs/phpstorm-url-handler@8f7a8df).