Merge pull request #47 from tommilligan/travis-ci

.travis.yml

@@ -1,12 +1,43 @@
-sudo: false
 language: rust
 cache: cargo
+dist: focal
+
+# this matrix generates jobs for the `test` stage
+os:
+  - linux
+  - windows
 rust:
   - nightly
   - beta
   - stable
   - 1.35.0
-script: |
-  cargo build &&
-  cargo test &&
-  cargo doc
+
+# run only one job first to see if it fails, then run the rest
+jobs:
+  include:
+    - stage: fast-test
+      os: linux
+      rust: stable
+      # additional behaviour just for this comprehensive job
+      before_install: |
+        rustup component add rustfmt clippy
+        cargo install cargo-audit cargo-tarpaulin
+      script: |
+        cargo fmt -- --check
+        cargo audit --deny warnings
+        cargo clippy --all-targets -- -D warnings
+        cargo test
+        cargo doc --no-deps
+      after_success: |
+        cargo tarpaulin --out xml
+        bash <(curl -s https://codecov.io/bash)
+  exclude:
+    - stage: test
+      os: linux
+      rust: stable
+  allow_failures:
+    - rust: nightly
+
+stages:
+  - fast-test
+  - test

CHANGELOG.md

@@ -0,0 +1,5 @@
+# Unreleased
+
+## Internal
+
+- Test Windows targets in CI (#46, @tommilligan)

Cargo.toml

@@ -1,7 +1,11 @@
 [package]
 name = "pretty_assertions"
 version = "0.6.1"
-authors = ["Colin Kiegel <kiegel@gmx.de>", "Florent Fayolle <florent.fayolle69@gmail.com>"]
+authors = [
+    "Colin Kiegel <kiegel@gmx.de>",
+    "Florent Fayolle <florent.fayolle69@gmail.com>",
+    "Tom Milligan <code@tommilligan.net>",
+]
 edition = "2018"
 
 description = "Overwrite `assert_eq!` and `assert_ne!` with drop-in replacements, adding colorful diffs."

README.md

@@ -71,6 +71,7 @@ use pretty_assertions::{assert_eq, assert_ne};
   a diff.
 * Under Windows, the terminal state is modified to properly handle VT100 
   escape sequences, which may break display for certain use cases.
+* The minimum supported rust version (MSRV) is 1.35.0
 
 ## License
 

src/format_changeset.rs

@@ -17,7 +17,7 @@ const SIGN_LEFT: char = '<'; // - < ←
 // Credits johannhof (MIT License)
 
 pub fn format_changeset(f: &mut fmt::Formatter, changeset: &Changeset) -> fmt::Result {
-    let ref diffs = changeset.diffs;
+    let diffs = &changeset.diffs;
 
     writeln!(
         f,
@@ -119,7 +119,7 @@ pub fn format_replacement(f: &mut dyn fmt::Write, added: &str, removed: &str) ->
             _ => (),
         }
     }
-    writeln!(f, "")?;
+    writeln!(f)?;
 
     // RIGHT side (==what's new)
     paint!(f, Green, "{}", SIGN_RIGHT)?;
@@ -145,7 +145,7 @@ pub fn format_replacement(f: &mut dyn fmt::Write, added: &str, removed: &str) ->
         }
     }
 
-    writeln!(f, "")
+    writeln!(f)
 }
 
 #[test]

tests/assert_ne.rs

@@ -1,3 +1,5 @@
+#![allow(clippy::eq_op)]
+
 #[allow(unused_imports)]
 use pretty_assertions::{assert_eq, assert_ne};
 #[test]