Skip to content

Commit 4014919

Browse files
committed
fix: try use a generator instead
Still figuring out what eslint is playing at.
1 parent 6a4bfc3 commit 4014919

File tree

1 file changed

+7
-10
lines changed

1 file changed

+7
-10
lines changed

packages/eslint-plugin-svelte/src/rules/derived-has-same-inputs-outputs.ts

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -97,20 +97,17 @@ export default createRule('derived-has-same-inputs-outputs', {
9797
loc: element.loc,
9898
messageId: 'unexpected',
9999
data: { name: expectedName },
100-
fix: (fixer) => {
100+
*fix(fixer) {
101101
const scope = getSourceCode(context).getScope(fn.body);
102102
const variable = scope.variables.find((variable) => variable.name === element.name);
103103

104-
if (!variable) {
105-
return fixer.replaceText(element, expectedName);
106-
}
104+
yield fixer.replaceText(element, expectedName);
107105

108-
return [
109-
fixer.replaceText(element, expectedName),
110-
...variable.references.map((ref) =>
111-
fixer.replaceText(ref.identifier, expectedName)
112-
)
113-
];
106+
if (variable) {
107+
for (const ref of variable.references) {
108+
yield fixer.replaceText(ref.identifier, expectedName);
109+
}
110+
}
114111
}
115112
});
116113
}

0 commit comments

Comments
 (0)