ci: test against MSRV and 1.87 (#857)

* pin rust versions in ci

* apply clippy suggestion

* kill python after benchmark

Author: lemmih

.github/workflows/ci.yaml

@@ -18,9 +18,8 @@ jobs:
       fail-fast: false
       matrix:
         rust:
-          - stable
+          - 1.87.0 # Stable release as of 2025-05-17
           - 1.70.0 # MSRV
-          - nightly
 
     steps:
       - uses: actions/checkout@v4

benches/benchmarks/external_process.rs

@@ -23,17 +23,19 @@ fn python_fibonacci(c: &mut Criterion) {
         .is_ok();
 
     if has_python3 {
-        let process = create_command()
+        let mut process = create_command()
             .stdin(Stdio::piped())
             .stdout(Stdio::piped())
             .spawn()
             .expect("Unable to start python process");
 
         let mut stdin = process
             .stdin
+            .take()
             .expect("Unable to get stdin for child process");
         let stdout = process
             .stdout
+            .take()
             .expect("Unable to get stdout for child process");
         let mut stdout = BufReader::new(stdout);
         c.bench_function("fibonacci-python", |b| {
@@ -51,6 +53,7 @@ fn python_fibonacci(c: &mut Criterion) {
         });
 
         // Ensure that your child process terminates itself gracefully!
+        process.kill().expect("Unable to kill child process");
     }
 }