- Notifications
You must be signed in to change notification settings - Fork45
Add support for discovery extensions#760
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
83825d3
toa9277d8
CompareUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
87a764c
to27e0ae6
CompareCurrently, the implementation requires returning the absolute path to aresource manifest - but I Right now, it expects JSON like this: {"resourceManifestPath":"/foo/bar/baz.dsc.resource.json"} But we could have it return this, and parse the last segment of the path to determine kind: {"absoluteManifestPath":"/foo/bar/baz.dsc.extension.json"} Or require it return either If we support extensions discovering extensions, we would need to recursively handle extension |
Uh oh!
There was an error while loading.Please reload this page.
I think we can defer extension discovery til when we need it |
Renamed from |
Uh oh!
There was an error while loading.Please reload this page.
f1be730
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
PR Summary
dsc
to list extensions:dsc extension list
discover
capability currently supported.The output from a discovery extension must match:
PR Context
Fix#681