-
Notifications
You must be signed in to change notification settings - Fork 910
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
Isolate more features #6854
Isolate more features #6854
Conversation
…contributions; make groupings the same
…erwidget contribution into their respective files
# Conflicts: # src/sql/workbench/parts/dataExplorer/browser/dataExplorer.contribution.ts # src/sql/workbench/parts/query/browser/queryActions.ts
# Conflicts: # src/sql/workbench/common/actions.contribution.ts # src/sql/workbench/common/actions.ts # src/sql/workbench/parts/dashboard/browser/widgets/explorer/explorerWidget.common.contribution.ts # src/sql/workbench/parts/dataExplorer/common/nodeActions.common.contribution.ts # src/sql/workbench/parts/objectExplorer/browser/serverTreeActionProvider.ts # src/sql/workbench/parts/query/browser/query.contribution.ts
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.
import 'sql/workbench/parts/dataExplorer/common/extensions.contribution'; | ||
|
||
//restore | ||
import 'sql/workbench/parts/restore/browser/restore.contribution'; |
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.
is there a reason we are leaving out the other contributions?
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.
Make sure you re-run the OE integration tests and update the context menu expected actions if needed
# Conflicts: # extensions/integration-tests/src/objectExplorer.test.ts
@Charles-Gagnon i fixed the tests for this pr |
Isolates backup/restore/notebooks and extension entry points to their own contribution files