-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[$500] Update the package "jest" to the latest version #9941
Comments
Triggered auto assignment to @greg-schroeder ( |
Ah yes, we should update the package "jest" to the latest version, of course |
Triggered auto assignment to @sketchydroide ( |
https://github.com/Expensify/Expensify/issues/218314 Getting a 404 not found page on this link. |
Not sure if this is supposed to be an |
Triggered auto assignment to @jboniface ( |
what is hard about it ? just upgrade, fix the breaking changes and make the tests pass ? |
@adelekennedy this needs an upwork job when you get a chance (i unassigned myself since you were assigned to manage this in the E/E repo) |
@sibelius nothing hard per se (hopefully), indeed it's just about:
|
Am I eligible for compensation here? I actually created the PR #10300. I can still upgrade this if needed. |
Do you think we should "cut our losses" and compensate you $500, or that you should continue investigating this for a while and we'll see later? |
I am happy to keep tracking the state of the fix on the reanimated repo. Currently, I am checking it on weekly basis. But I understand, that upgrading jest has no apparent advantage, a working version of jest should be fine. Any decision is fine with me. |
Ah, ok. I guess we can keep tracking for a little while and see what happens |
There is a new PR with the fix on |
@parasharrajat any update on this? |
The new PR is not yet merged software-mansion/react-native-reanimated#3559. |
@parasharrajat @sketchydroide I see the above PR is merged - from the above I think you're due for compensation from the work you've done - does this need to be stay open or can we pay and close? |
Thanks for the bump. Let me check the latest state of the PR and I will update you. |
@parasharrajat going through my issues before ooo, what are next steps? |
Tried the new version. There are a few errors. I will try to fix those. PR is updated to the latest. |
👋 just want to confirm what next steps look like here. @parasharrajat there are errors with the new version, which you will fix in your existing PR? Is that accurate? |
Yes, I will try to fix those and share an update as soon as possible. |
Tried a few things but had no luck. Will look more into this later. Or I will let you know if we should close this. |
This is not blocking anything ATM. I will share an update asap. Working on some priorities. |
any update @parasharrajat, just trying to keep this updated. |
let me just check it now. |
I tried to solve the issue but unfortunately, I have no solution to the following error. ● Test suite failed to run
TypeError: setImmediate is not a function
22 | }
23 |
> 24 | LogBox.ignoreLogs([
| ^
The latest changes can be seen here #10300. Now, the jest 29 version is available which has many changes so I think there is no advantage of upgrading to jest 28. For now, I think jest 26 is working great with the project and we don't need to upgrade as there are breaking changes. I suggest we drop this upgrade. |
That seems ok, we might as well upgrade to the next one, and it seems everything is still working, if we need we can open it againSent from my iPhoneOn 12 Dec 2022, at 19:15, Rajat Parashar ***@***.***> wrote:
I tried to solve the issue but unfortunately, I have no solution to the following error.
● Test suite failed to run
TypeError: setImmediate is not a function
22 | }
23 |
24 | LogBox.ignoreLogs([
| ^
setImmediate is directly used in RN and jsdom does not contain setImmediate. It is supported in node testEnvironment. But Tests will fail in node testEnvironment so we can't switch.
The latest changes can be seen here #10300.
Now, the jest 29 version is available which has many changes so I think there is no advantage of upgrading to jest 28.
For now, I think jest 26 is working great with the project and we don't need to upgrade as there are breaking changes.
I suggest we drop this upgrade.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
That seems ok, we might as well upgrade to the next one, and it seems everything is still working, if we need we can open it again |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
coming from this issue https://github.com/Expensify/Expensify/issues/218314
from @flodnv:
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: