Skip to content

Commit 4e54988

Browse files
mshanemcclaude
andcommitted
refactor: apply low-severity review findings - W-23094922
- eslint.config.mjs: document checkDirectories:false (v68 added dir checks) - plan: drop redundant "(alpha-ish ordering)" note Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
1 parent 87ad5df commit 4e54988

2 files changed

Lines changed: 2 additions & 1 deletion

File tree

.claude/plans/W-23094922.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424

2525
### Phase 1a — enable rule (config-only commit)
2626

27-
- Insert `'unicorn/prefer-hoisting-branch-code': 'error',` in unicorn block after `prefer-export-from`, before `prefer-includes` (alpha-ish ordering).
27+
- Insert `'unicorn/prefer-hoisting-branch-code': 'error',` in unicorn block after `prefer-export-from`, before `prefer-includes`.
2828
- Only file changed: `eslint.config.mjs`.
2929
- `npm run lint` here will surface the full violation set (used to scope 1b).
3030
- Commit (config lands independently even if 1b blocked): `chore: enable unicorn/prefer-hoisting-branch-code in eslint.config.mjs`

eslint.config.mjs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -211,6 +211,7 @@ export default [
211211
'error',
212212
{
213213
case: 'camelCase',
214+
// v68 added directory-name checks; preserve prior file-only behavior
214215
checkDirectories: false
215216
}
216217
],

0 commit comments

Comments
 (0)