-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[Config] Improve PHPdoc / IDE autocomplete #21047
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
@@ -142,7 +142,7 @@ public function enumPrototype() | |||
* If this function has been called and the node is not set during the finalization | |||
* phase, it's default value will be derived from its children default values. | |||
* | |||
* @return ArrayNodeDefinition | |||
* @return $this |
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.
Should probably fixed consistently (see also #20290).
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.
Happening :) lets see if any remain after #21054
I'd be in favor of moving this to a lower branch. |
Agree.. ill try rebasing 2.7 somewhere today. |
Thank you @ro0NL. |
This PR was merged into the 2.7 branch. Discussion ---------- [Config] Improve PHPdoc / IDE autocomplete | Q | A | ------------- | --- | Branch? | 2.7 | Bug fix? | yes-ish | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #... <!-- #-prefixed issue number(s), if any --> | License | MIT | Doc PR | symfony/symfony-docs#... <!--highly recommended for new features--> The missing pieces for fixing the autocomplete in IDE's (tested in `PhpStorm 2016.3.2`). Each method seems to be found now for the framework bundle configuration 😱 Actually uses #20923 where needed, but i think we should go with it consistently. Relates to #20290 (comment) Commits ------- 3f3c6e0 [Config] Improve PHPdoc / IDE autocomplete
…ee builder (Koc) This PR was merged into the 4.2 branch. Discussion ---------- [Config] Improve PHPdoc / IDE autocomplete for config tree builder | Q | A | ------------- | --- | Branch? | 4.2 | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | - | License | MIT | Doc PR | - follow up of #27476 and #21047 Commits ------- 21f7977 Improve PHPdoc / IDE autocomplete for config tree builder
The missing pieces for fixing the autocomplete in IDE's (tested in
PhpStorm 2016.3.2
).Each method seems to be found now for the framework bundle configuration 😱
Actually uses #20923 where needed, but i think we should go with it consistently.
Relates to #20290 (comment)