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

update node version #474

Closed
wants to merge 4 commits into from
Closed

update node version #474

wants to merge 4 commits into from

Conversation

mrbinky3000
Copy link
Collaborator

What:

I am updating the required node version to 20.10.0 and removing node-sass in favor of sass

Why:

The build and publish seems to be failing. I think it's because the version of node we were using is long dead and there were warnings about the version of node. We'll see if the fix is as simple as updating node.

How:

Simply bumped the version of node in .npmrc, uninstalled node-sass, ran true-level, and then installed sass. I ran build, test, lint, and started the app locally to see if it all still worked.

Checklist:

  • Documentation added/updated N/A
  • Typescript definitions updated N/A
  • Tests added and passing
  • Ready to be merged

@mrbinky3000 mrbinky3000 marked this pull request as draft October 25, 2024 18:46
@mrbinky3000
Copy link
Collaborator Author

We are going to update everything in next major release which is currently under development

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.

1 participant