Skip to content

Commit e491126

Browse files
authored
refactor(lint): upgrade some unsafe fixes to safe fixes (#6063)
1 parent 0b5e564 commit e491126

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

46 files changed

+505
-539
lines changed

.changeset/rule-fix.md

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
---
2+
"@biomejs/biome": minor
3+
---
4+
5+
Upgrade some unsafe fixes to safe fixes.
6+
7+
The following rules have now a safe fix:
8+
9+
- [noExtraBooleanCast](https://biomejs.dev/linter/rules/no-extra-boolean-cast)
10+
- [noNonoctalDecimalEscape](https://biomejs.dev/linter/rules/no-nonoctal-decimal-escape)
11+
- [noSwitchDeclarations](https://biomejs.dev/linter/rules/no-switch-declarations)
12+
- [noThisInStatic](https://biomejs.dev/linter/rules/no-this-in-static)
13+
- [noUnusedTemplateLiteral](https://biomejs.dev/linter/rules/no-unused-template-literal)
14+
- [noUselessContinue](https://biomejs.dev/linter/rules/no-useless-continue)
15+
- [noUselessElse](https://biomejs.dev/linter/rules/no-useless-else)
16+
- [noUselessStringConcat](https://biomejs.dev/linter/rules/no-useless-string-concat)
17+
- [useArrayLiterals](https://biomejs.dev/linter/rules/use-array-literals)
18+
- [useExponentiationOperator](https://biomejs.dev/linter/rules/use-exponentiation-operator)
19+
- [useNumberToFixedDigitsArgument](https://biomejs.dev/linter/rules/use-number-to-fixed-digits-argument)
20+
- [useNumericLiterals](https://biomejs.dev/linter/rules/use-numeric-literals)
21+
- [useSimplifiedLogicExpression](https://biomejs.dev/linter/rules/use-simplified-logic-expression)

crates/biome_js_analyze/src/lint/complexity/no_extra_boolean_cast.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ declare_lint_rule! {
6161
sources: &[RuleSource::Eslint("no-extra-boolean-cast")],
6262
recommended: true,
6363
severity: Severity::Information,
64-
fix_kind: FixKind::Unsafe,
64+
fix_kind: FixKind::Safe,
6565
}
6666
}
6767

crates/biome_js_analyze/src/lint/complexity/no_this_in_static.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ declare_lint_rule! {
8484
sources: &[RuleSource::EslintMysticatea("no-this-in-static")],
8585
recommended: true,
8686
severity: Severity::Warning,
87-
fix_kind: FixKind::Unsafe,
87+
fix_kind: FixKind::Safe,
8888
}
8989
}
9090

crates/biome_js_analyze/src/lint/complexity/no_useless_string_concat.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ declare_lint_rule! {
6868
sources: &[RuleSource::Eslint("no-useless-concat")],
6969
recommended: false,
7070
severity: Severity::Information,
71-
fix_kind: FixKind::Unsafe,
71+
fix_kind: FixKind::Safe,
7272
}
7373
}
7474

crates/biome_js_analyze/src/lint/complexity/no_useless_string_raw.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,6 @@ declare_lint_rule! {
3636
language: "js",
3737
recommended: true,
3838
severity: Severity::Information,
39-
severity: Severity::Information,
4039
}
4140
}
4241

crates/biome_js_analyze/src/lint/complexity/use_numeric_literals.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ declare_lint_rule! {
6161
sources: &[RuleSource::Eslint("prefer-numeric-literals")],
6262
recommended: true,
6363
severity: Severity::Warning,
64-
fix_kind: FixKind::Unsafe,
64+
fix_kind: FixKind::Safe,
6565
}
6666
}
6767

crates/biome_js_analyze/src/lint/complexity/use_simplified_logic_expression.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ declare_lint_rule! {
5252
language: "js",
5353
recommended: false,
5454
severity: Severity::Information,
55-
fix_kind: FixKind::Unsafe,
55+
fix_kind: FixKind::Safe,
5656
}
5757
}
5858

crates/biome_js_analyze/src/lint/correctness/no_nonoctal_decimal_escape.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ declare_lint_rule! {
6060
sources: &[RuleSource::Eslint("no-nonoctal-decimal-escape")],
6161
recommended: true,
6262
severity: Severity::Error,
63-
fix_kind: FixKind::Unsafe,
63+
fix_kind: FixKind::Safe,
6464
}
6565
}
6666

crates/biome_js_analyze/src/lint/correctness/no_switch_declarations.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ declare_lint_rule! {
7676
sources: &[RuleSource::Eslint("no-case-declarations")],
7777
recommended: true,
7878
severity: Severity::Error,
79-
fix_kind: FixKind::Unsafe,
79+
fix_kind: FixKind::Safe,
8080
}
8181
}
8282

crates/biome_js_analyze/src/lint/correctness/no_useless_continue.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ declare_lint_rule! {
7575
language: "js",
7676
recommended: true,
7777
severity: Severity::Error,
78-
fix_kind: FixKind::Unsafe,
78+
fix_kind: FixKind::Safe,
7979
}
8080
}
8181

0 commit comments

Comments
 (0)