Skip to content

Update gulp-sass to v6.0.1 to address deprecation warnings#112

Open
agibson-godaddy wants to merge 1 commit intomasterfrom
update-gulp-sass
Open

Update gulp-sass to v6.0.1 to address deprecation warnings#112
agibson-godaddy wants to merge 1 commit intomasterfrom
update-gulp-sass

Conversation

@agibson-godaddy
Copy link
Contributor

@agibson-godaddy agibson-godaddy commented Feb 5, 2026

Summary

This updates the gulp-sass dependency to 6.0.1, which fixes these errors:

Deprecation Warning [legacy-js-api]: The legacy JS API is deprecated and will be removed in Dart Sass 2.0.0.

Details

As per the release post the outputStyle option has been renamed to style.

QA

  1. Use this version of sake
  2. Use a plugin that has scss files (woocommerce-memberships will do)
  3. Run sake compile:scss
    • No "The legacy JS API is deprecated" errors
    • SCSS is compiled to CSS

@agibson-godaddy agibson-godaddy changed the title Update gulp-sass to v6.0.1 Update gulp-sass to v6.0.1 to address deprecation warnings Feb 5, 2026
@agibson-godaddy agibson-godaddy marked this pull request as ready for review February 5, 2026 14:49
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the gulp-sass dependency from version 5.1.0 to 6.0.1 to resolve deprecation warnings related to the legacy JS API in Dart Sass. The update requires a corresponding change to the Sass compilation configuration.

Changes:

  • Updated gulp-sass dependency to version 6.0.1
  • Renamed the outputStyle option to style in the Sass compilation task to align with the new API

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
package.json Updates gulp-sass dependency from ^5.1.0 to ^6.0.1
tasks/compile.js Changes Sass option from outputStyle to style to match the updated API

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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