CARVIEW |
Navigation Menu
-
-
Notifications
You must be signed in to change notification settings - Fork 9.7k
[DependencyInjection][Routing] Add JSON schema for validating and autocompleting YAML config files #61282
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
This comment was marked as resolved.
This comment was marked as resolved.
src/Symfony/Component/DependencyInjection/Loader/schema/dic/services/services-1.0.schema.json
Outdated
Show resolved
Hide resolved
Little bit off topic, but what do you think about a JsonSchema component? we could also reuse this in symfony/ai |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
not mine to write, that typically the kind of stuff I'd rather outsource to another lib |
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as 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.
I've noticed a few issues while testing.
src/Symfony/Component/DependencyInjection/Loader/schema/dic/services/services-1.0.schema.json
Outdated
Show resolved
Hide resolved
src/Symfony/Component/DependencyInjection/Loader/schema/dic/services/services-1.0.schema.json
Outdated
Show resolved
Hide resolved
src/Symfony/Component/DependencyInjection/Loader/schema/dic/services/services-1.0.schema.json
Outdated
Show resolved
Hide resolved
src/Symfony/Component/DependencyInjection/Loader/schema/dic/services/services-1.0.schema.json
Outdated
Show resolved
Hide resolved
src/Symfony/Component/DependencyInjection/Loader/schema/dic/services/services-1.0.schema.json
Outdated
Show resolved
Hide resolved
84382b0
to
0aaba40
Compare
PR updated. |
…ocompleting YAML config files
0aaba40
to
f535cc6
Compare
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.
Thank you @nicolas-grekas. |
Related to #59620
This provides autocompletion/validation on IDEs that support JSON schema for YAML files, such as phpStorm.
Note that this cannot support yaml tags, so no autocompletion after
!service
for example.The only way to provide this would be to ship a language server. Out of reach I fear.
Recipe update at symfony/recipes#1447