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

feat: add support for node-sass v9 #1140

Merged
merged 2 commits into from
May 22, 2023
Merged

feat: add support for node-sass v9 #1140

merged 2 commits into from
May 22, 2023

Conversation

snitin315
Copy link
Member

This PR contains a:

  • bugfix
  • new feature
  • code refactor
  • test update
  • typo fix
  • metadata update

Motivation / Use-Case

Breaking Changes

Additional Info

@codecov
Copy link

codecov bot commented May 22, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (13b0ae1) 94.66% compared to head (b229a22) 94.66%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1140   +/-   ##
=======================================
  Coverage   94.66%   94.66%           
=======================================
  Files           5        5           
  Lines         375      375           
  Branches      137      137           
=======================================
  Hits          355      355           
  Misses         17       17           
  Partials        3        3           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@alexander-akait alexander-akait marked this pull request as ready for review May 22, 2023 11:41
@alexander-akait alexander-akait merged commit 9a03c87 into master May 22, 2023
@alexander-akait alexander-akait deleted the feat/node-sass-9 branch May 22, 2023 11:42
@alexander-akait alexander-akait mentioned this pull request May 22, 2023
@u01jmg3
Copy link

u01jmg3 commented May 22, 2023

Thanks for the quick resolution. Does this need to be tweaked too?

"node": ">= 14.15.0"

@alexander-akait
Copy link
Member

No, other developers can use v8, v7 and etc, we can't break their projects

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