Skip to content

USWDS-Compile: Use gulp-sass v6 to use modern Sass JS API (Updated)#159

Closed
mdmower-csnw wants to merge 4 commits into
uswds:developfrom
mdmower-csnw:dw-mdm-gulp-sass-update
Closed

USWDS-Compile: Use gulp-sass v6 to use modern Sass JS API (Updated)#159
mdmower-csnw wants to merge 4 commits into
uswds:developfrom
mdmower-csnw:dw-mdm-gulp-sass-update

Conversation

@mdmower-csnw
Copy link
Copy Markdown
Contributor

@mdmower-csnw mdmower-csnw commented Dec 15, 2025

This is a continuation of #140 since additional movement is not expected there. Several changes in that PR were only formatting, which have since been applied in develop.

Summary

Update gulp-sass to v6.0.1 so that the new Sass v3 JS API can be used. See v6 release notes. This is a non breaking change for uswds-compile consumers because the options API is not exposed outside of boolean option sassDeprecationWarnings which is used to configure quietDeps.

See also migration guide from v5 to v6: https://www.npmjs.com/package/gulp-sass#migrating-to-version-6

Related issue

This is a step towards #133 but does not close that issue. A subsequent pull request can build off of this work to expose the modern options API to end users.

Problem statement

For #133, it would be best to avoid exposing the Sass options API until it is modernized, so that users do not initially configure with a deprecated API and then soon after be required to update options again for the modern API.

Solution

This pull request updates gulp-sass to the latest v6 revision and updates internal usage of the options API.

Testing and review

No observable differences should occur. This pull request is successful if no impact is noticed by consumers of uswds-compile.

Dependency updates

Dependency name Previous version New version
gulp-sass 5.1.0 6.0.1

@mdmower-csnw mdmower-csnw requested a review from a team as a code owner December 15, 2025 20:53
@mdmower-csnw mdmower-csnw changed the title Use gulp-sass v6 to use modern Sass JS API (Updated) USWDS-Compile: Use gulp-sass v6 to use modern Sass JS API (Updated) Dec 15, 2025
@mdmower-csnw
Copy link
Copy Markdown
Contributor Author

@annepetersen - Hi Anne, following up on your Slack post, you — yes, you! — can get involved, would now be a good time to nudge maintainers on this pull request, which would open the door for #145 as well?

@annepetersen
Copy link
Copy Markdown
Contributor

annepetersen commented Feb 11, 2026

Appreciate the nudge! This one in particular, probably not, but many others could use a hand.

For context, decisions and prioritization have headed in the direction of deprecating USWDS compile rather than expending effort here.

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