Skip to content

Conversation

TheIronDev
Copy link
Contributor

@TheIronDev TheIronDev commented May 23, 2022

Description

This commit adds a peerDep of >=jest@28 and [email protected].

jest@28 includes jestjs/jest#11961, which
supports exports in package.json.

Without this, jest users that use jest@27 or below will throw an error
when firebase-functions-test attempts to import several named exports.

Fixes #151

This commit adds a peerDep of >=jest@28.

jest@28 includes jestjs/jest#11961, which
supports `exports` in `package.json`.

Without this, jest users that use jest@27 or below will throw an error
when firebase-functions-test attempts to import several named exports.

Fixes #151
@TheIronDev TheIronDev requested a review from colerogers May 23, 2022 18:57
The original motivation to upgrade was to resolve the breaking change
mentioned [here](DefinitelyTyped/DefinitelyTyped#60310)

I chose this version to maintain parity with firebase-functions [here](https://github.com/firebase/firebase-functions/blob/master/package.json#L224)
This commit continues the chain of upgrading stale dependencies due
to a fun cascading effect of adding a peerDep.

If you give a moose a muffin...
@TheIronDev TheIronDev requested a review from bkendall May 23, 2022 21:47
@TheIronDev TheIronDev merged commit d49e709 into master May 24, 2022
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.

Cannot import firebase-functions/v2
2 participants