Skip to content

node 23 update, clean up#91

Closed
m1ga wants to merge 3 commits into1_Xfrom
node23
Closed

node 23 update, clean up#91
m1ga wants to merge 3 commits into1_Xfrom
node23

Conversation

@m1ga
Copy link
Contributor

@m1ga m1ga commented Apr 8, 2025

  • removed <18 parts
  • added node 23 to package.json
  • github action updates

I left the version number as it is, so you can move the branches around if you want to clean it up.

@cb1kenobi
Copy link
Contributor

As mentioned on Slack, I think we should abandon the 1_X branch. The old Node C++ API is painful to manage.

@m1ga
Copy link
Contributor Author

m1ga commented Apr 8, 2025

sure, sounds like the better plan. There was no request for it yet. I just had a Mac that had node 23 installed and saw the error of the missing file. I went back to 22 (LTS version) and it was fine. You Slack plan sounds like a better/cleaner way

@m1ga m1ga closed this Apr 8, 2025
@m1ga m1ga deleted the node23 branch April 8, 2025 13:52
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.

2 participants