Error if the NURSERY selector is used with preview

astral-sh · Jan 29, 2024 · 66ccfd3 · 66ccfd3
1 parent 04000a4
commit 66ccfd3
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 13 deletions.
diff --git a/crates/ruff/tests/integration_test.rs b/crates/ruff/tests/integration_test.rs
@@ -878,15 +878,12 @@ fn nursery_group_selector_preview_enabled() {
     assert_cmd_snapshot!(cmd
         .pass_stdin("I=42\n"), @r###"
     success: false
-    exit_code: 1
+    exit_code: 2
     ----- stdout -----
-    -:1:1: CPY001 Missing copyright notice at top of file
-    -:1:2: E225 [*] Missing whitespace around operator
-    Found 2 errors.
-    [*] 1 fixable with the `--fix` option.
 
     ----- stderr -----
-    warning: The `NURSERY` selector has been deprecated.
+    ruff failed
+      Cause: The `NURSERY` selector is deprecated and cannot be used with preview mode enabled.
     "###);
 }
 

diff --git a/crates/ruff_workspace/src/configuration.rs b/crates/ruff_workspace/src/configuration.rs
@@ -860,13 +860,10 @@ impl LintConfiguration {
             for (kind, selector) in selection.selectors_by_kind() {
                 #[allow(deprecated)]
                 if matches!(selector, RuleSelector::Nursery) {
-                    let suggestion = if preview.mode.is_disabled() {
-                        " Use the `--preview` flag instead."
-                    } else {
-                        // We have no suggested alternative since there is intentionally no "PREVIEW" selector
-                        ""
-                    };
-                    warn_user_once!("The `NURSERY` selector has been deprecated.{suggestion}");
+                    if preview.mode.is_enabled() {
+                        return Err(anyhow!("The `NURSERY` selector is deprecated and cannot be used with preview mode enabled."));
+                    }
+                    warn_user_once!("The `NURSERY` selector has been deprecated. Use the `--preview` flag instead.");
                 };
 
                 // Only warn for the following selectors if used to enable rules