Skip to content

chore(linter/no-unassigned-vars): move to category to correctness #12832

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 5, 2025
Merged

chore(linter/no-unassigned-vars): move to category to correctness #12832

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion ...ore-path fixtures__linter__.customignore --no-ignore fixtures__linter__nan.js@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ working directory:
help: Use the isNaN function to compare with NaN.

Found 1 warning and 0 errors.
Finished in <variable>ms on 1 file with 87 rules using 1 threads.
Finished in <variable>ms on 1 file with 88 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion ...pshots/_--ignore-pattern _____.js --ignore-pattern _____.vue fixtures__linter@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ arguments: --ignore-pattern **/*.js --ignore-pattern **/*.vue fixtures/linter
working directory:
----------
Found 0 warnings and 0 errors.
Finished in <variable>ms on 0 files with 87 rules using 1 threads.
Finished in <variable>ms on 0 files with 88 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion apps/oxlint/src/snapshots/_--import-plugin -A all -D no-cycle fixtures__flow__@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ arguments: --import-plugin -A all -D no-cycle fixtures/flow/
working directory:
----------
Found 0 warnings and 0 errors.
Finished in <variable>ms on 2 files with 89 rules using 1 threads.
Finished in <variable>ms on 2 files with 90 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion apps/oxlint/src/snapshots/_--import-plugin fixtures__flow__index.mjs@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ arguments: --import-plugin fixtures/flow/index.mjs
working directory:
----------
Found 0 warnings and 0 errors.
Finished in <variable>ms on 1 file with 89 rules using 1 threads.
Finished in <variable>ms on 1 file with 90 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion ..._vitest_replace__eslintrc.json fixtures__eslintrc_vitest_replace__foo.test.js@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ working directory:
help: Remove the appending `.skip`

Found 1 warning and 1 error.
Finished in <variable>ms on 1 file with 99 rules using 1 threads.
Finished in <variable>ms on 1 file with 100 rules using 1 threads.
----------
CLI result: LintFoundErrors
----------
2 changes: 1 addition & 1 deletion apps/oxlint/src/snapshots/_-D correctness fixtures__linter__debugger.js@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ working directory:
help: Remove the debugger statement

Found 0 warnings and 1 error.
Finished in <variable>ms on 1 file with 87 rules using 1 threads.
Finished in <variable>ms on 1 file with 88 rules using 1 threads.
----------
CLI result: LintFoundErrors
----------
2 changes: 1 addition & 1 deletion ...nt/src/snapshots/_-W correctness -A no-debugger fixtures__linter__debugger.js@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ arguments: -W correctness -A no-debugger fixtures/linter/debugger.js
working directory:
----------
Found 0 warnings and 0 errors.
Finished in <variable>ms on 1 file with 86 rules using 1 threads.
Finished in <variable>ms on 1 file with 87 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion ... fixtures__eslintrc_env__eslintrc_no_env.json fixtures__eslintrc_env__test.js@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ working directory:
`----

Found 1 warning and 0 errors.
Finished in <variable>ms on 1 file with 88 rules using 1 threads.
Finished in <variable>ms on 1 file with 89 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion ..._-W no-undef -c fixtures__no_undef__eslintrc.json fixtures__no_undef__test.js@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ working directory:
`----

Found 1 warning and 0 errors.
Finished in <variable>ms on 1 file with 88 rules using 1 threads.
Finished in <variable>ms on 1 file with 89 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion ...e_extension__eslintrc.json fixtures__config_ignore_patterns__ignore_extension@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ working directory:
help: Delete this file or add some code to it.

Found 1 warning and 0 errors.
Finished in <variable>ms on 1 file with 87 rules using 1 threads.
Finished in <variable>ms on 1 file with 88 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion ...ures__eslintrc_env__eslintrc_env_browser.json fixtures__eslintrc_env__test.js@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ arguments: -c fixtures/eslintrc_env/eslintrc_env_browser.json fixtures/eslintrc_
working directory:
----------
Found 0 warnings and 0 errors.
Finished in <variable>ms on 1 file with 88 rules using 1 threads.
Finished in <variable>ms on 1 file with 89 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion ...ots/_-c fixtures__eslintrc_off__eslintrc.json fixtures__eslintrc_off__test.js@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ working directory:
help: Delete this file or add some code to it.

Found 1 warning and 0 errors.
Finished in <variable>ms on 1 file with 87 rules using 1 threads.
Finished in <variable>ms on 1 file with 88 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion ..._vitest_replace__eslintrc.json fixtures__eslintrc_vitest_replace__foo.test.js@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ arguments: -c fixtures/eslintrc_vitest_replace/eslintrc.json fixtures/eslintrc_v
working directory:
----------
Found 0 warnings and 0 errors.
Finished in <variable>ms on 1 file with 87 rules using 1 threads.
Finished in <variable>ms on 1 file with 88 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion ...c/snapshots/_-c fixtures__linter__eslintrc.json fixtures__linter__debugger.js@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ working directory:
help: Remove the debugger statement

Found 0 warnings and 1 error.
Finished in <variable>ms on 1 file with 87 rules using 1 threads.
Finished in <variable>ms on 1 file with 88 rules using 1 threads.
----------
CLI result: LintFoundErrors
----------
2 changes: 1 addition & 1 deletion ..._-c fixtures__no_console_off__eslintrc.json fixtures__no_console_off__test.js@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ arguments: -c fixtures/no_console_off/eslintrc.json fixtures/no_console_off/test
working directory:
----------
Found 0 warnings and 0 errors.
Finished in <variable>ms on 1 file with 87 rules using 1 threads.
Finished in <variable>ms on 1 file with 88 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion ...atch__eslintrc.json -W no-empty fixtures__no_empty_allow_empty_catch__test.js@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ arguments: -c fixtures/no_empty_allow_empty_catch/eslintrc.json -W no-empty fixt
working directory:
----------
Found 0 warnings and 0 errors.
Finished in <variable>ms on 1 file with 88 rules using 1 threads.
Finished in <variable>ms on 1 file with 89 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion ...h__eslintrc.json -W no-empty fixtures__no_empty_disallow_empty_catch__test.js@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ working directory:
help: Remove this block or add a comment inside it

Found 1 warning and 0 errors.
Finished in <variable>ms on 1 file with 88 rules using 1 threads.
Finished in <variable>ms on 1 file with 89 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
6 changes: 3 additions & 3 deletions ...test.ts -c fixtures__overrides__.oxlintrc.json fixtures__overrides__other.jsx@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ working directory:
help: Replace var with let or const

Found 0 warnings and 1 error.
Finished in <variable>ms on 1 file with 89 rules using 1 threads.
Finished in <variable>ms on 1 file with 90 rules using 1 threads.
----------
CLI result: LintFoundErrors
----------
Expand All @@ -42,7 +42,7 @@ working directory:
help: Delete this console statement.

Found 1 warning and 1 error.
Finished in <variable>ms on 1 file with 89 rules using 1 threads.
Finished in <variable>ms on 1 file with 90 rules using 1 threads.
----------
CLI result: LintFoundErrors
----------
Expand All @@ -61,7 +61,7 @@ working directory:
help: Replace var with let or const

Found 0 warnings and 1 error.
Finished in <variable>ms on 1 file with 89 rules using 1 threads.
Finished in <variable>ms on 1 file with 90 rules using 1 threads.
----------
CLI result: LintFoundErrors
----------
2 changes: 1 addition & 1 deletion ...napshots/_-c fixtures__overrides__directories-config.json fixtures__overrides@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ working directory:
help: Remove the debugger statement

Found 2 warnings and 2 errors.
Finished in <variable>ms on 7 files with 86 rules using 1 threads.
Finished in <variable>ms on 7 files with 87 rules using 1 threads.
----------
CLI result: LintFoundErrors
----------
2 changes: 1 addition & 1 deletion ...slintrc.json --disable-typescript-plugin fixtures__typescript_eslint__test.ts@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ working directory:
`----

Found 1 warning and 1 error.
Finished in <variable>ms on 1 file with 50 rules using 1 threads.
Finished in <variable>ms on 1 file with 51 rules using 1 threads.
----------
CLI result: LintFoundErrors
----------
2 changes: 1 addition & 1 deletion ...xtures__typescript_eslint__eslintrc.json fixtures__typescript_eslint__test.ts@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ working directory:
`----

Found 2 warnings and 1 error.
Finished in <variable>ms on 1 file with 62 rules using 1 threads.
Finished in <variable>ms on 1 file with 63 rules using 1 threads.
----------
CLI result: LintFoundErrors
----------
2 changes: 1 addition & 1 deletion apps/oxlint/src/snapshots/_fixtures__astro__debugger.astro@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ working directory:
help: Remove the debugger statement

Found 4 warnings and 0 errors.
Finished in <variable>ms on 1 file with 87 rules using 1 threads.
Finished in <variable>ms on 1 file with 88 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion apps/oxlint/src/snapshots/_fixtures__linter@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ working directory:
help: Use the isNaN function to compare with NaN.

Found 3 warnings and 0 errors.
Finished in <variable>ms on 3 files with 87 rules using 1 threads.
Finished in <variable>ms on 3 files with 88 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion .../oxlint/src/snapshots/_fixtures__linter__debugger.js fixtures__linter__nan.js@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ working directory:
help: Use the isNaN function to compare with NaN.

Found 2 warnings and 0 errors.
Finished in <variable>ms on 2 files with 87 rules using 1 threads.
Finished in <variable>ms on 2 files with 88 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion apps/oxlint/src/snapshots/_fixtures__linter__debugger.js@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ working directory:
help: Remove the debugger statement

Found 1 warning and 0 errors.
Finished in <variable>ms on 1 file with 87 rules using 1 threads.
Finished in <variable>ms on 1 file with 88 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion apps/oxlint/src/snapshots/_fixtures__linter__js_as_jsx.js@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ working directory:
help: Remove the debugger statement

Found 1 warning and 0 errors.
Finished in <variable>ms on 1 file with 87 rules using 1 threads.
Finished in <variable>ms on 1 file with 88 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
22 changes: 20 additions & 2 deletions apps/oxlint/src/snapshots/_fixtures__svelte__debugger.svelte@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,26 @@ working directory:
`----
help: Remove the debugger statement

Found 1 warning and 0 errors.
Finished in <variable>ms on 1 file with 87 rules using 1 threads.
! ]8;;https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-unassigned-vars.html\eslint(no-unassigned-vars)]8;;\: 'title' is always 'undefined' because it's never assigned.
,-[fixtures/svelte/debugger.svelte:4:13]
3 |
4 | export let title;
: ^^^^^
5 | export let person;
`----
help: Variable declared without assignment. Either assign a value or remove the declaration.

! ]8;;https://oxc.rs/docs/guide/usage/linter/rules/eslint/no-unassigned-vars.html\eslint(no-unassigned-vars)]8;;\: 'person' is always 'undefined' because it's never assigned.
,-[fixtures/svelte/debugger.svelte:5:13]
4 | export let title;
5 | export let person;
: ^^^^^^
6 |
`----
help: Variable declared without assignment. Either assign a value or remove the declaration.

Found 3 warnings and 0 errors.
Finished in <variable>ms on 1 file with 88 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion apps/oxlint/src/snapshots/_fixtures__vue__debugger.vue@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ working directory:
help: Remove the debugger statement

Found 2 warnings and 0 errors.
Finished in <variable>ms on 1 file with 87 rules using 1 threads.
Finished in <variable>ms on 1 file with 88 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion apps/oxlint/src/snapshots/_fixtures__vue__empty.vue@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ arguments: fixtures/vue/empty.vue
working directory:
----------
Found 0 warnings and 0 errors.
Finished in <variable>ms on 1 file with 87 rules using 1 threads.
Finished in <variable>ms on 1 file with 88 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion apps/oxlint/src/snapshots/_foo.asdf@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ arguments: foo.asdf
working directory:
----------
Found 0 warnings and 0 errors.
Finished in <variable>ms on 0 files with 87 rules using 1 threads.
Finished in <variable>ms on 0 files with 88 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion ...snapshots/fixtures__config_ignore_patterns__ignore_directory_-c eslintrc.json@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ working directory: fixtures/config_ignore_patterns/ignore_directory
help: Delete this file or add some code to it.

Found 1 warning and 0 errors.
Finished in <variable>ms on 1 file with 87 rules using 1 threads.
Finished in <variable>ms on 1 file with 88 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion ...ore_patterns__with_oxlintrc_-c .__test__eslintrc.json --ignore-pattern _.ts .@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ working directory: fixtures/config_ignore_patterns/with_oxlintrc
help: Delete this file or add some code to it.

Found 1 warning and 0 errors.
Finished in <variable>ms on 1 file with 87 rules using 1 threads.
Finished in <variable>ms on 1 file with 88 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
4 changes: 2 additions & 2 deletions ...lias_rules_-c .oxlintrc-eslint.json test.js -c .oxlintrc-unicorn.json test.js@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ arguments: -c .oxlintrc-eslint.json test.js
working directory: fixtures/disable_eslint_and_unicorn_alias_rules
----------
Found 0 warnings and 0 errors.
Finished in <variable>ms on 1 file with 50 rules using 1 threads.
Finished in <variable>ms on 1 file with 51 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
Expand All @@ -16,7 +16,7 @@ arguments: -c .oxlintrc-unicorn.json test.js
working directory: fixtures/disable_eslint_and_unicorn_alias_rules
----------
Found 0 warnings and 0 errors.
Finished in <variable>ms on 1 file with 63 rules using 1 threads.
Finished in <variable>ms on 1 file with 64 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion ...pshots/fixtures__extends_config_--config extends_rules_config.json console.js@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ working directory: fixtures/extends_config
help: Delete this console statement.

Found 0 warnings and 1 error.
Finished in <variable>ms on 1 file with 88 rules using 1 threads.
Finished in <variable>ms on 1 file with 89 rules using 1 threads.
----------
CLI result: LintFoundErrors
----------
2 changes: 1 addition & 1 deletion ...xtends_config_--config relative_paths__extends_extends_config.json console.js@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ working directory: fixtures/extends_config
help: Delete this console statement.

Found 0 warnings and 1 error.
Finished in <variable>ms on 1 file with 88 rules using 1 threads.
Finished in <variable>ms on 1 file with 89 rules using 1 threads.
----------
CLI result: LintFoundErrors
----------
2 changes: 1 addition & 1 deletion apps/oxlint/src/snapshots/fixtures__extends_config_--disable-nested-config@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ working directory: fixtures/extends_config
help: Remove the debugger statement

Found 3 warnings and 0 errors.
Finished in <variable>ms on 4 files with 87 rules using 1 threads.
Finished in <variable>ms on 4 files with 88 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
4 changes: 2 additions & 2 deletions apps/oxlint/src/snapshots/fixtures__ignore_file_current_dir_ .@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ arguments:
working directory: fixtures/ignore_file_current_dir
----------
Found 0 warnings and 0 errors.
Finished in <variable>ms on 0 files with 87 rules using 1 threads.
Finished in <variable>ms on 0 files with 88 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
Expand All @@ -16,7 +16,7 @@ arguments: .
working directory: fixtures/ignore_file_current_dir
----------
Found 0 warnings and 0 errors.
Finished in <variable>ms on 0 files with 87 rules using 1 threads.
Finished in <variable>ms on 0 files with 88 rules using 1 threads.
----------
CLI result: LintSucceeded
----------
2 changes: 1 addition & 1 deletion ...lint/src/snapshots/fixtures__import-cycle_--import-plugin -D import__no-cycle@oxlint.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ working directory: fixtures/import-cycle
-> ./b - fixtures/import-cycle/b.ts

Found 0 warnings and 2 errors.
Finished in <variable>ms on 2 files with 90 rules using 1 threads.
Finished in <variable>ms on 2 files with 91 rules using 1 threads.
----------
CLI result: LintFoundErrors
----------
Loading
Loading