removes notify and includes gulp-dart-scss

uswds / public-sans

A strong, neutral, principles-driven, open source typeface for text or display

https://public-sans.digital.gov/

Other

4.46k stars 102 forks source link

removes notify and includes gulp-dart-scss #209

Closed scottqueen-bixal closed 3 years ago

scottqueen-bixal commented 3 years ago

gulp notify is not really a needed dependency
move all dependencies to devDependecies
replace gulp-sass with gulp-dart-sass

removes yargs-parser and trim-newlines vul

scottqueen-bixal commented 3 years ago

@mejiaj can you take a look at this? got rid of the gulp-sass wrapper and replaced with gulp-dart-scss as we have in other repos - look ok? did I miss anything obvious?

mejiaj commented 3 years ago

@gscottqueen LGTM