Skip to content

unbreak Node.js 20#495

Closed
legobeat wants to merge 3 commits into
LavaMoat:mainfrom
legobeat:unbreak-node-18
Closed

unbreak Node.js 20#495
legobeat wants to merge 3 commits into
LavaMoat:mainfrom
legobeat:unbreak-node-18

Conversation

@legobeat

@legobeat legobeat commented May 5, 2023

Copy link
Copy Markdown
Collaborator

#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

legobeat commented May 5, 2023

Copy link
Copy Markdown
Collaborator Author

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
Comment thread packages/node/package.json Outdated
@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
Copy Markdown
Collaborator Author

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

@boneskull

boneskull commented Aug 25, 2023

Copy link
Copy Markdown
Member

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
Copy Markdown
Collaborator 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.

3 participants