-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
Jest 21 Master Task #4332
Comments
I think the inline require one is best handled by people more familiar with Jest internals at the state it is. It works (the PR that is) by excluding a given module, but that's a slippery slope (and easy to break by messing with e.g. Happy to help out though, of course! (link: #3786) And personally I'd like to see a solution for EDIT: I'm senile, Aaron already said what I said, only better.
EDIT2: #4340 |
To finish the batch of work on pretty-format:
While we are fixing edge cases with breaking changes:
But let #3429 marinate until following version of Jest:
|
Do we need #4278 for Jest 21? |
@rogeliog I'm not sure, I think there is a ton of stuff we could do here to make the API better and unclear right now if we should touch it and make it a thing for 21 at all. Let's discuss in the jest-core channel. |
Just making sure, but #4091 will land with |
@wtgtybhertgeghgtwtg yup, it's already on master (from which the releases are cut) |
it's actually already on |
I think #4074 and browser support documentation.... : ) What about the failing CI? Does anybody have an update on that? |
CI is only failing on AppVeyor with a minor issue that nobody has spent time on fixing yet. |
In the PR comments you mentioned that #3660 will land in Jest 21 as well. I finished up the PR as per your comments so hopefully it can be merged in time for Jest 21:) |
Yep, it's part of 21! |
Also, should we use a milestone for Jest 21? |
Usage in the browser also needs docs, and probably a full example |
@evan-scott-zocdoc this is not an issue for requests. If you want an issue to be fixed, send a PR, and it will be merged when it is ready. |
We are going to punt on some further fixes, as they aren't breaking changes. We'll publish 21 soon. |
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. |
jest-matchers
toexpect
.displayName
to MPR. Add displayName to MPR #4327The text was updated successfully, but these errors were encountered: