From 1ca3063ac38132f6596122e9e108cf4390ae9bff Mon Sep 17 00:00:00 2001
From: Stefan Holderbach <sholderbach@users.noreply.github.com>
Date: Mon, 2 May 2022 11:32:57 +0200
Subject: [PATCH] Fix CI to run doctests again (#5410)

The faster `cargo nextest` currently doesn't support running the doctests.

Thus, add an additional step for them with cargo's default test runner.

- Fix doctests for the `nu-pretty-hex` crate
---
 .github/workflows/ci.yml        | 23 ++++++++++++++---------
 crates/nu-pretty-hex/src/lib.rs |  6 +++---
 2 files changed, 17 insertions(+), 12 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 6cb64f0c0f..1b5ac25b8f 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -17,9 +17,9 @@ jobs:
           - stable
         include:
           - style: all
-            flags: '--all-features'
+            flags: "--all-features"
           - style: default
-            flags: ''
+            flags: ""
         exclude:
           - platform: windows-latest
             style: default
@@ -41,7 +41,7 @@ jobs:
 
       - uses: Swatinem/rust-cache@v1
         with:
-            key: ${{ matrix.style }}v1 # increment this to bust the cache if needed
+          key: ${{ matrix.style }}v1 # increment this to bust the cache if needed
 
       - name: Rustfmt
         uses: actions-rs/cargo@v1
@@ -61,7 +61,6 @@ jobs:
           command: clippy
           args: --workspace ${{ matrix.flags }} -- -D warnings -D clippy::unwrap_used -A clippy::needless_collect
 
-
   test:
     strategy:
       fail-fast: false
@@ -72,9 +71,9 @@ jobs:
           - stable
         include:
           - style: all
-            flags: '--all-features'
+            flags: "--all-features"
           - style: default
-            flags: ''
+            flags: ""
         exclude:
           - platform: windows-latest
             style: default
@@ -96,7 +95,7 @@ jobs:
 
       - uses: Swatinem/rust-cache@v1
         with:
-            key: ${{ matrix.style }}v1 # increment this to bust the cache if needed
+          key: ${{ matrix.style }}v1 # increment this to bust the cache if needed
 
       - uses: taiki-e/install-action@nextest
 
@@ -106,6 +105,12 @@ jobs:
           command: nextest
           args: run --all ${{ matrix.flags }}
 
+      - name: Doctests
+        uses: actions-rs/cargo@v1
+        with:
+          command: test
+          args: --workspace --doc ${{ matrix.flags }}
+
   python-virtualenv:
     strategy:
       fail-fast: false
@@ -130,7 +135,7 @@ jobs:
 
       - uses: Swatinem/rust-cache@v1
         with:
-            key: "1" # increment this to bust the cache if needed
+          key: "1" # increment this to bust the cache if needed
 
       - name: Install Nushell
         uses: actions-rs/cargo@v1
@@ -176,7 +181,7 @@ jobs:
           toolchain: ${{ matrix.rust }}
           override: true
 
-      # This job does not use rust-cache because 1) we have limited cache space, 2) even 
+      # This job does not use rust-cache because 1) we have limited cache space, 2) even
       # without caching, it's not the slowest job. Revisit if those facts change.
 
       - name: Build nu_plugin_example
diff --git a/crates/nu-pretty-hex/src/lib.rs b/crates/nu-pretty-hex/src/lib.rs
index d34542a7ee..02a8d553f3 100644
--- a/crates/nu-pretty-hex/src/lib.rs
+++ b/crates/nu-pretty-hex/src/lib.rs
@@ -8,7 +8,7 @@
 //!
 //! ## Example of `simple_hex()`
 //! ```
-//! use pretty_hex::*;
+//! use nu_pretty_hex::*;
 //!
 //! let v = vec![222, 173, 190, 239, 202, 254, 32, 24];
 //! # #[cfg(feature = "alloc")]
@@ -23,7 +23,7 @@
 //! ```
 //! ## Example of `pretty_hex()`
 //! ```
-//! use pretty_hex::*;
+//! use nu_pretty_hex::*;
 //!
 //! let v = &include_bytes!("../tests/data");
 //! # #[cfg(feature = "alloc")]
@@ -40,7 +40,7 @@
 //! ```
 //! ## Example of `config_hex()`
 //! ```
-//! use pretty_hex::*;
+//! use nu_pretty_hex::*;
 //!
 //! let cfg = HexConfig {title: false, width: 8, group: 0, ..HexConfig::default() };
 //!