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

Sandbox: Update to Node.js 22.10 and NPM 10.9 #863

Closed
wants to merge 2 commits into from
Closed

Conversation

amotl
Copy link
Member

@amotl amotl commented Oct 25, 2024

About

Just maintenance: Update sandbox environment from previous Node.js 20 LTS to use the most recent Node.js 22 LTS.

@kneth
Copy link
Collaborator

kneth commented Oct 25, 2024

It looks like we have to wait for sass/node-sass#3418 to be merged (if it ever will be fixed).

@amotl
Copy link
Member Author

amotl commented Oct 25, 2024

Thanks. After removing node-sass from the list of dependencies, the build on CI completes successfully. Because this feels too easy, it would need manual verification that nothing goes south before merging. Can you quickly give it a spin next time, @msbt?

@amotl
Copy link
Member Author

amotl commented Oct 28, 2024

Update: This patch is incomplete. That other one intends to implement the update successfully. Thanks again, @msbt.

@amotl
Copy link
Member Author

amotl commented Oct 28, 2024

Superseded by GH-864, closing.

@amotl amotl closed this Oct 28, 2024
@amotl amotl deleted the nodejs-22.10 branch October 28, 2024 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants