Clarify the error for the wrong number of positional arguments

Closes #776
sass · Jul 25, 2019 · d840b84 · d840b84
1 parent d408a59
commit d840b84
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,9 @@
 
 * Avoid double `Error:` headers when reporting selector parse errors.
 
+* Clarify the error message when the wrong number of positional arguments are
+  passed along with a named argument.
+
 ## 1.22.8
 
 ### JavaScript API

diff --git a/lib/src/ast/sass/argument_declaration.dart b/lib/src/ast/sass/argument_declaration.dart
@@ -81,6 +81,7 @@ class ArgumentDeclaration implements SassNode {
 
     if (positional > arguments.length) {
       throw SassScriptException("Only ${arguments.length} "
+          "${names.isEmpty ? '' : 'positional '}"
           "${pluralize('argument', arguments.length)} allowed, but "
           "${positional} ${pluralize('was', positional, plural: 'were')} "
           "passed.");