Merge pull request #894 from gwenn/no_color

Check NO_COLOR environment variable

Author: gwenn

src/config.rs

@@ -149,9 +149,15 @@ impl Config {
 
     /// Tell if colors should be enabled.
     ///
-    /// By default, they are except if stdout is not a TTY.
+    /// By default, they are except if stdout is not a TTY or NO_COLOR
+    /// environment variable is set.
     #[must_use]
     pub fn color_mode(&self) -> ColorMode {
+        if self.color_mode == ColorMode::Enabled
+            && std::env::var_os("NO_COLOR").is_some_and(|os| !os.is_empty())
+        {
+            return ColorMode::Disabled;
+        }
         self.color_mode
     }
 

src/line_buffer.rs

@@ -878,7 +878,7 @@ impl LineBuffer {
         true
     }
 
-    /// Replaces the content between [`start`..`end`] with `text`
+    /// Replaces the content between `range` with `text`
     /// and positions the cursor to the end of text.
     pub fn replace<C: ChangeListener>(&mut self, range: Range<usize>, text: &str, cl: &mut C) {
         let start = range.start;