jest-config shouldn't import getType from matcher-utils

[thymikee]

See: #3527 (comment)

[wyze]

I could pick this up if you want @thymikee. If it shouldn't use getType should it just use a typeof to get the type? Or just use what is specified by Flow?

[thymikee]

I think you could even copy the whole function for now. We just don't want jest-config to be dependent on rather unrelated jest-matcher-utils library.

@cpojer what do you think about extracting similar shared helper functions into jest-helpers or jest-shared package?

[cpojer]

This one should be jest-get-type or something? ;)

[thymikee]

We're OK with having a package for a single function?

[cpojer]

yes, this is JavaScript.

[thymikee]

sindresorhus style, yea

[wyze]

Okay, coming right up!

[wyze]

I was going through and replacing all the instances of getType with the new package and came across this function: https://github.com/facebook/jest/blob/master/packages/jest-mock/src/index.js#L95

It looks very similar, but different enough that I left it alone for now.

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.