Add error when --no-doc is passed

Author: Urhengulas

src/bootstrap/src/core/config/flags.rs

@@ -428,6 +428,11 @@ pub enum Subcommand {
         #[arg(long)]
         /// Ignore `//@ ignore-backends` directives.
         bypass_ignore_backends: bool,
+
+        /// Deprecated. Use `--all-targets` or `--tests` instead.
+        #[arg(long)]
+        #[doc(hidden)]
+        no_doc: bool,
     },
     /// Build and run some test suites *in Miri*
     Miri {
@@ -447,6 +452,11 @@ pub enum Subcommand {
         /// Only run unit and integration tests
         #[arg(long)]
         tests: bool,
+
+        /// Deprecated. Use `--all-targets` or `--tests` instead.
+        #[arg(long)]
+        #[doc(hidden)]
+        no_doc: bool,
     },
     /// Build and run some benchmarks
     Bench {
@@ -577,6 +587,13 @@ impl Subcommand {
         }
     }
 
+    pub fn no_doc(&self) -> bool {
+        match *self {
+            Subcommand::Test { no_doc, .. } | Subcommand::Miri { no_doc, .. } => no_doc,
+            _ => false,
+        }
+    }
+
     pub fn bless(&self) -> bool {
         match *self {
             Subcommand::Test { bless, .. } => bless,

src/bootstrap/src/lib.rs

@@ -450,6 +450,10 @@ impl Build {
     ///
     /// By default all build output will be placed in the current directory.
     pub fn new(mut config: Config) -> Build {
+        if config.cmd.no_doc() {
+            panic!("DEPRECATED: `--no-doc` is deprecated. Use `--all-targets` or `--tests`.")
+        }
+
         let src = config.src.clone();
         let out = config.out.clone();