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

Not working with Node 19.1 #148

Closed
kyrylkov opened this issue Nov 27, 2022 · 7 comments
Closed

Not working with Node 19.1 #148

kyrylkov opened this issue Nov 27, 2022 · 7 comments

Comments

@kyrylkov
Copy link

For us croner stops working with Node 19.1 on Windows and Linux

@Hexagon
Copy link
Owner

Hexagon commented Nov 27, 2022

Verified. Node 19.1 added to test matrix, and fails catastrophically.

Thanks for the report, will have a look asap.

@Hexagon
Copy link
Owner

Hexagon commented Nov 27, 2022

@kyrylkov

Node 19.1 is broken by a bug in upstream V8, the problem is fixed in 19.0, but still exist in 19.1. Nothing to do about this except to wait for a new release of node,

nodejs/node#45171

19.1 - Broken

image

19.0 - OK

image

@kyrylkov
Copy link
Author

@Hexagon Yep, that's what I suspected. There was nothing in 19.1 vs 19.0 that could have broken it except a bug. Upcoming 19.2 should address this taking into account that it was fixed in upstream V8.

Thanks!

@Hexagon
Copy link
Owner

Hexagon commented Nov 27, 2022

Working my way around the Node 19.1 bug

@Hexagon Hexagon reopened this Nov 27, 2022
@Hexagon
Copy link
Owner

Hexagon commented Nov 27, 2022

Fixed in 5.3.5 👍

@Hexagon Hexagon closed this as completed Nov 27, 2022
@kibertoad
Copy link

@Hexagon This was fixed in 19.2.0, would it make sense to remove workaround now?

@Hexagon
Copy link
Owner

Hexagon commented Jan 4, 2023

@kibertoad Good point! The fix is just replacing narrow no break space with a normal space. Can keep it until we're sure that no one uses 19.1. If it isnt fixed in 19.1 too? Gotta check that. Always nice keeping the code as clean as possible 👍

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

No branches or pull requests

3 participants