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

Jest 21 Master Task #4332

Closed
6 tasks done
cpojer opened this issue Aug 23, 2017 · 18 comments
Closed
6 tasks done

Jest 21 Master Task #4332

cpojer opened this issue Aug 23, 2017 · 18 comments

Comments

@cpojer
Copy link
Member

cpojer commented Aug 23, 2017

@SimenB
Copy link
Member

SimenB commented Aug 23, 2017

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. Array.prototype, although that's not new with a lazy Jest). Something should be done WRT ensuring the env in a test is unable to touch the runner's env.

Happy to help out though, of course!

(link: #3786)

And personally I'd like to see a solution for jest-matchers to work in IE10, not sure how high if a priority that is, though?

EDIT: I'm senile, Aaron already said what I said, only better.

i messed around with this issue this week. My opinion that we should make everything that is not a test execute in the outer VM scope.

EDIT2: #4340

@pedrottimark
Copy link
Contributor

pedrottimark commented Aug 23, 2017

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:

  • get feedback about the concept
  • make sure it works with multi-line strings in backticks
  • formats diff when snapshot test fails as similarly as possible

@rogeliog
Copy link
Contributor

Do we need #4278 for Jest 21?

@cpojer
Copy link
Member Author

cpojer commented Aug 24, 2017

@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.

@wtgtybhertgeghgtwtg
Copy link
Contributor

Just making sure, but #4091 will land with jest@21 and friends, right?

@thymikee
Copy link
Collaborator

@wtgtybhertgeghgtwtg yup, it's already on master (from which the releases are cut)

@thymikee
Copy link
Collaborator

it's actually already on jest@test if you need it

@skovhus
Copy link
Contributor

skovhus commented Aug 24, 2017

I think #4074 and browser support documentation.... : )

What about the failing CI? Does anybody have an update on that?

@cpojer
Copy link
Member Author

cpojer commented Aug 24, 2017

CI is only failing on AppVeyor with a minor issue that nobody has spent time on fixing yet.

@excitement-engineer
Copy link
Contributor

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:)

@cpojer
Copy link
Member Author

cpojer commented Aug 27, 2017

Yep, it's part of 21!

@rogeliog
Copy link
Contributor

@cpojer #4379

@rogeliog
Copy link
Contributor

Also, should we use a milestone for Jest 21?

@SimenB
Copy link
Member

SimenB commented Aug 28, 2017

Usage in the browser also needs docs, and probably a full example

@evan-scott-zocdoc
Copy link

@cpojer #4100? :)

@cpojer
Copy link
Member Author

cpojer commented Aug 29, 2017

@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.

@cpojer
Copy link
Member Author

cpojer commented Sep 4, 2017

We are going to punt on some further fixes, as they aren't breaking changes. We'll publish 21 soon.

@cpojer cpojer closed this as completed Sep 4, 2017
@github-actions
Copy link

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.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

9 participants