Skip to content

Commit 45cfa00

Browse files
committed
chore: remove 'matches upstream' comments — describe behavior directly
1 parent 5101d33 commit 45cfa00

1 file changed

Lines changed: 5 additions & 6 deletions

File tree

lib/rules/template-require-form-method.js

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,8 @@ function parseConfig(config) {
2727
}
2828

2929
// Invalid configuration (e.g. unknown method in allowedMethods). Throw a
30-
// descriptive error to surface the problem to the user, matching upstream
31-
// ember-template-lint (which throws instead of silently disabling).
30+
// descriptive error to surface the problem to the user rather than silently
31+
// disabling the rule.
3232
throw new Error(
3333
'template-require-form-method: invalid configuration. Expected one of:\n' +
3434
' * boolean - `true` to enable / `false` to disable\n' +
@@ -91,10 +91,9 @@ module.exports = {
9191
},
9292

9393
create(context) {
94-
// Match upstream ember-template-lint: when no options are provided,
95-
// parseConfig(undefined) returns false and the rule is disabled. This
96-
// rule must be enabled explicitly via `true` or an `allowedMethods`
97-
// object.
94+
// When no options are provided, parseConfig(undefined) returns false and
95+
// the rule is disabled. It must be enabled explicitly via `true` or an
96+
// `allowedMethods` object.
9897
const rawOption = context.options[0];
9998
const config = parseConfig(rawOption);
10099

0 commit comments

Comments
 (0)