Merge branch 'main' into binops

CITATION.cff

@@ -0,0 +1,21 @@
+abstract: Find inadequately-tested code that can be removed without any tests failing.
+authors:
+  - alias: sourcefrog
+    family-names: Pool
+    given-names: Martin
+cff-version: 1.2.0
+date-released: 2024-01-13
+keywords:
+  - Rust
+  - cargo
+  - coverage
+  - development-tools::testing
+  - mutants
+  - mutation-testing
+  - testing
+license: MIT
+message: Please cite this project using these information.
+repository-code: https://github.com/sourcefrog/cargo-mutants
+title: cargo-mutants
+url: https://mutants.rs/
+version: 24.1.0

Cargo.toml

@@ -162,3 +162,10 @@ targets = [
 [profile.dist]
 inherits = "release"
 lto = "thin"
+
+# Config for <https://github.com/crate-ci/cargo-release/blob/master/docs/reference.md>
+[workspace.metadata.release]
+pre-release-replacements = [
+    { "file" = "NEWS.md", search = "## Unreleased", replace = "$0\n ## {{version}}", exactly = 1 },
+    { "file" = "CITATION.cff", search = "^version: .*", replace = "version: {{version}}", exactly = 1 },
+]

NEWS.md

@@ -2,16 +2,16 @@
 
 ## Unreleased
 
+- New: Mutate `+, -, *, /, %, &, ^, |, <<, >>` binary ops, and their corresponding assignment ops like `+=`.
+
+- Changed: Stop generating mutations of `||` and `&&` to `!=` and `||`, because it seems to raise too many low-value false positives that may be hard to test.
+
 ## 24.1.0
 
 - New! `cargo mutants --test-tool nextest`, or `test_tool = "nextest"` in `.cargo/mutants.toml` runs tests under [Nextest](https://nexte.st/). Some trees have tests that only work under Nextest, and this allows them to be tested. In other cases Nextest may be significantly faster, because it will exit soon after the first test failure.
 
 - Fixed: Fixed spurious "Patch input contains repeated filenames" error when `--in-diff` is given a patch that deletes multiple files.
 
-- New: Mutate `+, -, *, /, %, &, ^, |, <<, >>` binary ops, and their corresponding assignment ops like `+=`.
-
-- Changed: Stop generating mutations of `||` and `&&` to `!=` and `||`, because it seems to raise too many low-value false positives that may be hard to test.
-
 ## 23.12.2
 
 - New: A `--shard k/n` allows you to split the work across n independent parallel `cargo mutants` invocations running on separate machines to get a faster overall solution on large suites. You, or your CI system, are responsible for launching all the shards and checking whether any of them failed.

src/console.rs

@@ -555,7 +555,6 @@ impl CopyModel {
     /// Update that some bytes have been copied.
     ///
     /// `bytes_copied` is the total bytes copied so far.
-    #[allow(dead_code)]
     fn bytes_copied(&mut self, bytes_copied: u64) {
         self.bytes_copied = bytes_copied
     }

src/copy_tree.rs

@@ -80,7 +80,7 @@ pub fn copy_tree(
             })?;
             total_bytes += bytes_copied;
             total_files += 1;
-            console.copy_progress(bytes_copied);
+            console.copy_progress(total_bytes);
         } else if ft.is_dir() {
             std::fs::create_dir_all(&dest_path)
                 .with_context(|| format!("Failed to create directory {dest_path:?}"))?;