release: v1.9.0 (#3794)

biomejs · Sep 12, 2024 · 894c3c5 · 894c3c5
1 parent 9c9a599
commit 894c3c5
Show file tree

Hide file tree

Showing 366 changed files with 5,150 additions and 4,545 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -9,7 +9,7 @@ New entries must be placed in a section entitled `Unreleased`.
 Read
 our [guidelines for writing a good changelog entry](https://github.com/biomejs/biome/blob/main/CONTRIBUTING.md#changelog).
 
-## Unreleased
+## v1.9.0 (2024-09-12)
 
 ### Analyzer
 
@@ -287,6 +287,62 @@ our [guidelines for writing a good changelog entry](https://github.com/biomejs/b
 
 ### Linter
 
+#### Promoted rules
+
+New rules are incubated in the nursery group.
+Once stable, we promote them to a stable group.
+
+The following CSS rules are promoted:
+
+- [a11y/useGenericFontNames](https://biomejs.dev/linter/rules/use-generic-font-names/)
+- [correctness/noInvalidDirectionInLinearGradient](https://biomejs.dev/linter/rules/no-invalid-direction-in-linear-gradient/)
+- [correctness/noInvalidGridAreas](https://biomejs.dev/linter/rules/no-invalid-grid-areas/)
+- [correctness/noInvalidPositionAtImportRule](https://biomejs.dev/linter/rules/no-invalid-position-at-import-rule/)
+- [correctness/noUnknownFunction](https://biomejs.dev/linter/rules/no-unknown-function/)
+- [correctness/noUnknownMediaFeatureName](https://biomejs.dev/linter/rules/no-unknown-media-feature-name/)
+- [correctness/noUnknownProperty](https://biomejs.dev/linter/rules/no-unknown-property/)
+- [correctness/noUnknownUnit](https://biomejs.dev/linter/rules/no-unknown-unit/)
+- [correctness/noUnmatchableAnbSelector](https://biomejs.dev/linter/rules/no-unmatchable-anb-selector/)
+- [suspicious/noDuplicateAtImportRules](https://biomejs.dev/linter/rules/no-duplicate-at-import-rules/)
+- [suspicious/noDuplicateFontNames](https://biomejs.dev/linter/rules/no-duplicate-font-names/)
+- [suspicious/noDuplicateSelectorsKeyframeBlock](https://biomejs.dev/linter/rules/no-duplicate-selectors-keyframe-block/)
+- [suspicious/noEmptyBlock](https://biomejs.dev/linter/rules/no-empty-block/)
+- [suspicious/noImportantInKeyframe](https://biomejs.dev/linter/rules/no-important-in-keyframe/)
+- [suspicious/noShorthandPropertyOverrides](https://biomejs.dev/linter/rules/no-shorthand-property-overrides/)
+
+The following JavaScript rules are promoted:
+
+- [a11y/noLabelWithoutControl](https://biomejs.dev/linter/rules/no-label-without-control/)
+- [a11y/useFocusableInteractive](https://biomejs.dev/linter/rules/use-focusable-interactive/)
+- [a11y/useSemanticElements](https://biomejs.dev/linter/rules/use-semantic-elements/)
+- [complexity/noUselessStringConcat](https://biomejs.dev/linter/rules/no-useless-string-concat/)
+- [complexity/noUselessUndefinedInitialization](https://biomejs.dev/linter/rules/no-useless-undefined-initialization/)
+- [complexity/useDateNow](https://biomejs.dev/linter/rules/use-date-now/)
+- [correctness/noUndeclaredDependencies](https://biomejs.dev/linter/rules/no-undeclared-dependencies/)
+- [correctness/noInvalidBuiltinInstantiation](https://biomejs.dev/linter/rules/no-invalid-builtin-instantiation/)
+- [correctness/noUnusedFunctionParameters](https://biomejs.dev/linter/rules/no-unused-function-parameters/)
+- [correctness/useImportExtensions](https://biomejs.dev/linter/rules/use-import-extensions/)
+- [performance/useTopLevelRegex](https://biomejs.dev/linter/rules/use-top-level-regex/)
+- [style/noDoneCallback](https://biomejs.dev/linter/rules/no-done-callback/)
+- [style/noYodaExpression](https://biomejs.dev/linter/rules/no-yoda-expression/)
+- [style/useConsistentBuiltinInstantiation](https://biomejs.dev/linter/rules/use-consistent-builtin-instantiation/)
+- [style/useDefaultSwitchClause](https://biomejs.dev/linter/rules/use-default-switch-clause/)
+- [style/useExplicitLengthCheck](https://biomejs.dev/linter/rules/use-explicit-length-check/)
+- [style/useThrowNewError](https://biomejs.dev/linter/rules/use-throw-new-error/)
+- [style/useThrowOnlyError](https://biomejs.dev/linter/rules/use-throw-only-error/)
+- [suspicious/noConsole](https://biomejs.dev/linter/rules/no-console/)
+- [suspicious/noEvolvingTypes](https://biomejs.dev/linter/rules/no-evolving-types/)
+- [suspicious/noMisplacedAssertion](https://biomejs.dev/linter/rules/no-misplaced-assertion/)
+- [suspicious/noReactSpecificProps](https://biomejs.dev/linter/rules/no-react-specific-props/)
+- [suspicious/useErrorMessage](https://biomejs.dev/linter/rules/use-error-message/)
+- [suspicious/useNumberToFixedDigitsArgument](https://biomejs.dev/linter/rules/use-number-to-fixed-digits-argument/)
+
+#### Deprecated rules
+
+- `correctness/noInvalidNewBuiltin` is deprecated. Use [correctness/noInvalidBuiltinInstantiation](https://biomejs.dev/linter/rules/no-invalid-builtin-instantiation/) instead.
+- `style/useSingleCaseStatement` is deprecated. Use [correctness/noSwitchDeclarations](https://biomejs.dev/linter/rules/no-switch-declarations/) instead.
+- `suspicious/noConsoleLog` is deprecated. Use [suspicious/noConsole](https://biomejs.dev/linter/rules/no-console/) instead.
+
 #### New features
 
 - Implement [css suppression action](https://github.com/biomejs/biome/issues/3278). Contributed by @togami2864

diff --git a/crates/biome_cli/src/commands/mod.rs b/crates/biome_cli/src/commands/mod.rs
@@ -416,7 +416,7 @@ pub enum BiomeCommand {
         sub_command: Option<MigrateSubCommand>,
     },
 
-    /// [EXPERIMENTAL] Searches for Grit patterns across a project.
+    /// EXPERIMENTAL: Searches for Grit patterns across a project.
     ///
     /// Note: GritQL escapes code snippets using backticks, but most shells
     /// interpret backticks as command invocations. To avoid this, it's best to

diff --git a/crates/biome_cli/src/execute/migrate/eslint_any_rule_to_biome.rs b/crates/biome_cli/src/execute/migrate/eslint_any_rule_to_biome.rs
diff --git a/crates/biome_cli/src/execute/migrate/eslint_eslint.rs b/crates/biome_cli/src/execute/migrate/eslint_eslint.rs
@@ -539,9 +539,9 @@ pub struct NoConsoleOptions {
     /// Allowed calls on the console object.
     pub allow: Vec<String>,
 }
-impl From<NoConsoleOptions> for biome_js_analyze::lint::nursery::no_console::NoConsoleOptions {
+impl From<NoConsoleOptions> for biome_js_analyze::lint::suspicious::no_console::NoConsoleOptions {
     fn from(val: NoConsoleOptions) -> Self {
-        biome_js_analyze::lint::nursery::no_console::NoConsoleOptions { allow: val.allow }
+        biome_js_analyze::lint::suspicious::no_console::NoConsoleOptions { allow: val.allow }
     }
 }
 

diff --git a/crates/biome_cli/src/execute/migrate/eslint_to_biome.rs b/crates/biome_cli/src/execute/migrate/eslint_to_biome.rs
@@ -210,7 +210,7 @@ fn migrate_eslint_rule(
         eslint_eslint::Rule::NoConsole(conf) => {
             if migrate_eslint_any_rule(rules, &name, conf.severity(), opts, results) {
                 if let eslint_eslint::RuleConf::Option(severity, rule_options) = conf {
-                    let group = rules.nursery.get_or_insert_with(Default::default);
+                    let group = rules.suspicious.get_or_insert_with(Default::default);
                     group.no_console = Some(biome_config::RuleFixConfiguration::WithOptions(
                         biome_config::RuleWithFixOptions {
                             level: severity.into(),

diff --git a/...me_cli/tests/snapshots/main_cases_handle_css_files/should_lint_files_by_when_enabled.snap b/...me_cli/tests/snapshots/main_cases_handle_css_files/should_lint_files_by_when_enabled.snap
@@ -30,7 +30,7 @@ lint ━━━━━━━━━━━━━━━━━━━━━━━━━
 # Emitted Messages
 
 ```block
-input.css:1:6 lint/nursery/noEmptyBlock ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+input.css:1:6 lint/suspicious/noEmptyBlock ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
   × An empty block isn't allowed.
   

diff --git a/...ome_cli/tests/snapshots/main_cases_handle_css_files/should_not_lint_files_by_default.snap b/...ome_cli/tests/snapshots/main_cases_handle_css_files/should_not_lint_files_by_default.snap
@@ -30,7 +30,7 @@ lint ━━━━━━━━━━━━━━━━━━━━━━━━━
 # Emitted Messages
 
 ```block
-input.css:1:6 lint/nursery/noEmptyBlock ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+input.css:1:6 lint/suspicious/noEmptyBlock ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
   × An empty block isn't allowed.
   

diff --git a/crates/biome_cli/tests/snapshots/main_commands_check/config_recommended_group.snap b/crates/biome_cli/tests/snapshots/main_commands_check/config_recommended_group.snap
@@ -37,16 +37,14 @@ check ━━━━━━━━━━━━━━━━━━━━━━━━
 # Emitted Messages
 
 ```block
-check.js:1:1 lint/correctness/noInvalidNewBuiltin  FIXABLE  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+check.js:1:1 lint/correctness/noInvalidBuiltinInstantiation  FIXABLE  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 
-  × Symbol cannot be called as a constructor.
+  × Use Symbol() instead of new Symbol().
   
   > 1 │ new Symbol("");
       │ ^^^^^^^^^^^^^^
   
-  i Calling Symbol with the new operator throws a TypeError.
-  
-  i Unsafe fix: Remove new.
+  i Unsafe fix: Remove new keyword.
   
     1 │ new·Symbol("");
       │ ----