cleanup: make it pass cargo clippy: for now allow Into without From

// allow it, because I am not sure we need it?
// better to explicitly turn into the newtype types
// but i might be wrong
// also for now, allowing it to pass `cargo clippy`

Author: alehander92

.github/workflows/rust.yml

@@ -10,7 +10,15 @@ env:
   CARGO_TERM_COLOR: always
 
 jobs:
-  build:
+  lint:
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: actions/checkout@v4
+    - name: Lint
+      run: cargo clippy
+
+  build-and-test:
 
     runs-on: ubuntu-latest
 

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "runtime_tracing"
-version = "0.5.14-experimental-5"
+version = "0.5.14-experimental-7"
 edition = "2021"
 
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

src/lib.rs

@@ -1,3 +1,8 @@
+// allow it, because I am not sure we need it?
+// better to explicitly turn into the newtype types
+// but i might be wrong
+// also for now, allowing it to pass `cargo clippy`
+#![allow(clippy::from_over_into)]
 mod tracer;
 mod types;
 pub use crate::tracer::{Tracer, NONE_TYPE_ID, NONE_VALUE};

src/tracer.rs

@@ -119,7 +119,7 @@ impl Tracer {
 
     pub fn register_step(&mut self, path: &Path, line: Line) {
         let path_id = self.ensure_path_id(path);
-        self.events.push(TraceLowLevelEvent::Step(StepRecord { path_id, line: line }));
+        self.events.push(TraceLowLevelEvent::Step(StepRecord { path_id, line}));
     }
 
     pub fn register_call(&mut self, function_id: FunctionId, args: Vec<FullValueRecord>) {