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: migrate from @salesforce/core v6 -> v7 #5556
Conversation
…estSetup in imports
…core when undefined
a6515e0
to
45c561a
Compare
…spaceContextUtil.test.ts
…ontextUtil.test.ts
@@ -4,6 +4,7 @@ | |||
* Licensed under the BSD 3-Clause license. | |||
* For full license text, see LICENSE.txt file in the repo root or https://opensource.org/licenses/BSD-3-Clause | |||
*/ | |||
import { ListMetadataQuery } from '@jsforce/jsforce-node/lib/api/metadata'; |
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.
@daphne-sfdc can you check to see if this is exposed at the module level?
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.
@@ -23,10 +23,10 @@ export const stopTelemetry = (): Promise<void> => { | |||
return Promise.resolve(); | |||
}; | |||
|
|||
export interface TelemetryModelJson extends JsonMap { | |||
export type TelemetryModelJson = JsonMap & { |
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.
@daphne-sfdc I don't believe JsonMap is necessary.
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.
@peternhale The double negative means you think it is necessary and what's currently there is fine? Or did you actually mean JSONMap is not necessary?
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.
not necessary
@@ -5,12 +5,12 @@ | |||
* For full license text, see LICENSE.txt file in the repo root or https://opensource.org/licenses/BSD-3-Clause | |||
*/ | |||
|
|||
import { DescribeSObjectResult } from '@jsforce/jsforce-node'; |
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.
We already moved to jsforce-node! 🍠
What does this PR do?
Migrates @salesforce/core from v6.7.4 -> v7.3.1
This upgrade means we are able to support Core v7 in our other dependencies, so the following dependencies are also migrated from Core v6 -> v7 in this PR:
Additional changes:
@salesforce/core/testSetup
to use top-level importsworkspaceContextUtils.test.ts
unit test to make it passWhat issues does this PR fix or reference?
@W-15472197@
Functionality Before
Extensions monorepo uses v6 of @salesforce/core
Functionality After
Extensions monrepo uses v7 of @salesforce/core, which means we are now able to pull in new features from our shared libraries