-
Notifications
You must be signed in to change notification settings - Fork 312
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
Modify DevToolsExtensionConfig model to handle static extensions #7602
Conversation
@@ -4,7 +4,12 @@ | |||
* **Breaking change:** rename `ExtensionsApi.extensionRootPathPropertyName` | |||
to `ExtensionsApi.packageRootUriPropertyName`, and modify the String value | |||
for the parameter from 'rootPath' to 'packageRootUri'. | |||
|
|||
* **Breaking change:** add new required JSON fields "devtoolsOptionsUri" and |
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.
How are we planning on handling these breaking changes for existing extensions in the ecosystem?
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.
These breaking changes are for the DevTools --> DevTools server API and shouldn't be called directly by DevTools extensions. The devtools_extensions
and devtools_app_shared
packages will be bumped to this new devtools_shared
version in their own major version bumps.
Work towards #7569