-
Notifications
You must be signed in to change notification settings - Fork 135
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
feat: use stacks schema merger #1770
Conversation
a8d7806
to
5676649
Compare
3d75cae
to
5ccc168
Compare
2edc244
to
a82961b
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.
Nice work! I left some suggestions
2c2ca5a
to
f7ccf0a
Compare
f7ccf0a
to
5337243
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.
Thanks for addressing the feedback
* chore: make provider schema available to schema merger * refactor: move PreloadEmbeddedSchema job into Defer as future jobs in Defer will depend on it * feat: make module meta available in schema merger * chore: add changie entry * fix: combine readers into one, address review feedback * refactor: continue parsing if loading stack component sources failed * Bump terraform-schema to `9fc1ce5`
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. |
Merge all the schemas (for stacks stack files) 💯
This brings support for completions for
inputs
andproviders
incomponent
blocks and for showing descriptions on hover among other schema related features. Note: this currently only supports local components with a source in a relative directory.