Skip to content

Commit da76f93

Browse files
committed
1-based line numbers in error messages
1 parent f1e7037 commit da76f93

File tree

6 files changed

+10
-10
lines changed

6 files changed

+10
-10
lines changed

fluent-dedent/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ let messages = ftl`
4141
welcome = Welcome, {$userName}!
4242
`;
4343

44-
// → RangeError("Insufficient indentation in line 1.")
44+
// → RangeError("Insufficient indentation in line 2.")
4545
```
4646

4747
Content must start on a new line and must end on a line of its own. The

fluent-dedent/src/index.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ export default function ftl(strings, ...values) {
3535
}
3636
if (lineIndent !== commonIndent) {
3737
// The indentation of the line must match commonIndent exacty.
38-
throw new RangeError(`Insufficient indentation in line ${idx}.`);
38+
throw new RangeError(`Insufficient indentation in line ${idx + 1}.`);
3939
}
4040
// Strip commonIndent.
4141
return line.slice(commonIndent.length);

fluent-dedent/test/blank_test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ suite("blank lines", function() {
4949
() => ftl`
5050
5151
`,
52-
/Insufficient indentation in line 0/
52+
/Insufficient indentation in line 1/
5353
);
5454
});
5555

fluent-dedent/test/content_test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ bar
5050
foo
5151
bar
5252
`,
53-
/Insufficient indentation in line 1/
53+
/Insufficient indentation in line 2/
5454
);
5555
});
5656
});

fluent-dedent/test/mixed_test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ suite("mixed indent", function() {
2727
() => ftl`
2828
\tfoo
2929
\t `,
30-
/Insufficient indentation in line 0/
30+
/Insufficient indentation in line 1/
3131
);
3232
});
3333
});

fluent-dedent/test/tabs_test.js

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ suite("tab indent", function() {
2727
() => ftl`
2828
\tfoo
2929
\t\t`,
30-
/Insufficient indentation in line 0/
30+
/Insufficient indentation in line 1/
3131
);
3232
});
3333

@@ -36,7 +36,7 @@ suite("tab indent", function() {
3636
() => ftl`
3737
\t\tfoo
3838
`,
39-
/Insufficient indentation in line 0/
39+
/Insufficient indentation in line 1/
4040
);
4141
});
4242

@@ -45,7 +45,7 @@ suite("tab indent", function() {
4545
() => ftl`
4646
foo
4747
\t\t`,
48-
/Insufficient indentation in line 0/
48+
/Insufficient indentation in line 1/
4949
);
5050
});
5151

@@ -54,7 +54,7 @@ suite("tab indent", function() {
5454
() => ftl`
5555
\t\tfoo
5656
`,
57-
/Insufficient indentation in line 0/
57+
/Insufficient indentation in line 1/
5858
);
5959
});
6060

@@ -63,7 +63,7 @@ suite("tab indent", function() {
6363
() => ftl`
6464
foo
6565
\t\t`,
66-
/Insufficient indentation in line 0/
66+
/Insufficient indentation in line 1/
6767
);
6868
});
6969
});

0 commit comments

Comments
 (0)