sass/dart-sass

Support globs on the command line

Open

#616 geöffnet am 6. März 2019

Auf GitHub ansehen
 (13 Kommentare) (10 Reaktionen) (0 zugewiesene Personen)Dart (4.192 Stars) (376 Forks)batch import
CLIenhancementhelp wanted

Beschreibung

Latest Sass on both Windows & OSX I get the following error

video.js@7.5.0 build:css:cdn: sass src/css/vjs-cdn.scss dist/alt/video-js-cdn.css "--watch" "'src/**/**/*.scss'"

This command works fine as-is in node-sass, but errors out in dart-sass. I would expect only a warning & ignoring the (unused) parameter.

Removing the parameter works well.

Perhaps the project I'm working on is unusual; older codebase but doesn't use grunt/gulp, but uses CLI commands in package.json (which is more of a current trend).

Contributor Guide

Tech Stack
Keine
Domain
cli
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 days
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Basic knowledge of CLIUnderstanding of glob patterns
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40
Research-Richtung
Investigate how dart sass currently handles command line file arguments. Look at the argument parser in the source code (likely in the bin entry point). The issue mentions that glob patterns passed as a string cause an error. Research how node sass implements glob support and consider using a glob library compatible with Dart. Check existing comments for any maintainer suggestions or rejected approaches.