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

unbreak Node.js 20 #495

Closed
wants to merge 3 commits into from
Closed

Conversation

legobeat
Copy link
Contributor

@legobeat legobeat commented May 5, 2023

#487 (run ci on node v18 and v20) unearthed some regressions. This aims to resolve that.

@legobeat legobeat requested a review from kumavis May 5, 2023 11:38
@legobeat
Copy link
Contributor Author

legobeat commented May 5, 2023

Next error doesn't seem as easy as bumping deps

@legobeat legobeat force-pushed the unbreak-node-18 branch 3 times, most recently from 23f1488 to f6c6400 Compare May 6, 2023 02:44
packages/node/package.json Outdated Show resolved Hide resolved
@legobeat legobeat force-pushed the unbreak-node-18 branch 5 times, most recently from 1a5e7a8 to de4ff79 Compare May 11, 2023 08:42
@legobeat legobeat changed the title unbreak Node.js 18 unbreak Node.js 18 and 20 May 11, 2023
@legobeat legobeat changed the title unbreak Node.js 18 and 20 unbreak Node.js 20 May 11, 2023
@legobeat legobeat force-pushed the unbreak-node-18 branch 2 times, most recently from 48959a1 to ff0501e Compare May 11, 2023 10:31
@legobeat legobeat force-pushed the unbreak-node-18 branch 4 times, most recently from 18bfd5b to adb8be0 Compare May 13, 2023 00:49
@legobeat
Copy link
Contributor Author

Removed unrelated commits which have been folded into other currently open PRs.

@boneskull
Copy link
Contributor

boneskull commented Aug 25, 2023

Since builds aren't kept forever, here's a gist of the failures.

UPDATE Maybe that's something else entirely. I'm not sure what's going on in that test. It's from my rebased copy of this branch which should have removed the engines restrictions...

@legobeat
Copy link
Contributor Author

Superseded by #718

@legobeat legobeat closed this Sep 14, 2023
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

Successfully merging this pull request may close these issues.

None yet

3 participants