Skip to content
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

chore: update deps #238

Merged
merged 2 commits into from Jul 19, 2021
Merged

chore: update deps #238

merged 2 commits into from Jul 19, 2021

Conversation

abdulsattar
Copy link
Contributor

Update deps

"fast-glob": "^3.2.5",
"jest": "26.6.3",
"fast-glob": "^3.2.7",
"jest": "27.0.6",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI, this will require a major version bump.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any way to support both jest 26 and 27, like we do in lwc-test?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Packages in lwc-test are jest plugins while sfdx-lwc-jest is a jest wrapper. I don't think it's possible with the way sfdx-lwc-jest is set up to distribute multiple versions of jest.

An alternative approach would be to externalize jest dependency and require all the SFDX projects to install jest as a peer dependency. In general Salesforce developers aren't familiar enough with the NodeJS ecosystem and NPM to deal with peer dependencies. Based on my experience Salesforce developers are looking for packaged and easy-to-install solutions.

@nolanlawson Are you concerned about the potential breaking changes jest@27 introduces?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pmdartus Based on the release notes it seems to me that there are things that can start breaking in v27. As long as we release a major version bump we should be fine. I didn't realize this was a wrapper and not a plugin.

"eslint": "^7.28.0",
"husky": "^5.2.0",
"eslint": "^7.30.0",
"husky": "^7.0.1",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@abdulsattar abdulsattar merged commit fc77345 into master Jul 19, 2021
@abdulsattar abdulsattar deleted the abdulsattar/update-deps-jul12 branch July 19, 2021 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants