sass/dart-sass

Support globs on the command line

Open

#616 opened on Mar 6, 2019

View on GitHub
 (13 comments) (10 reactions) (0 assignees)Dart (376 forks)batch import
CLIenhancementhelp wanted

Repository metrics

Stars
 (4,192 stars)
PR merge metrics
 (Avg merge 8d 8h) (18 merged PRs in 30d)

Description

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

Research direction
Inspect the CLI argument parsing in dart sass to understand why glob patterns are not expanded and where to add glob support.
Tech stack
dart
Domain
cli
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
DartSass
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70