Skip to content

Fix error scoping / error propagation #162

Fix error scoping / error propagation

Fix error scoping / error propagation #162

Triggered via push August 2, 2024 12:58
Status Success
Total duration 2m 23s
Artifacts

format-check.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

6 warnings
style
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
style
Unexpected input(s) 'dprint-version', valid inputs are ['entryPoint', 'args']
struct `Datum` has a public `len` method, but no `is_empty` method: crates/db-wiredtiger/src/bindings/cursor.rs#L67
warning: struct `Datum` has a public `len` method, but no `is_empty` method --> crates/db-wiredtiger/src/bindings/cursor.rs:67:5 | 67 | pub fn len(&self) -> usize { | ^^^^^^^^^^^^^^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_without_is_empty = note: `#[warn(clippy::len_without_is_empty)]` on by default
this `let...else` may be rewritten with the `?` operator: crates/kernel/src/vm/moo_frame.rs#L274
warning: this `let...else` may be rewritten with the `?` operator --> crates/kernel/src/vm/moo_frame.rs:274:9 | 274 | / let Some(scope) = self.scope_stack.pop() else { 275 | | return None; 276 | | }; | |__________^ help: replace it with: `let scope = self.scope_stack.pop()?;` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#question_mark = note: `#[warn(clippy::question_mark)]` on by default
struct `Datum` has a public `len` method, but no `is_empty` method: crates/db-wiredtiger/src/bindings/cursor.rs#L67
warning: struct `Datum` has a public `len` method, but no `is_empty` method --> crates/db-wiredtiger/src/bindings/cursor.rs:67:5 | 67 | pub fn len(&self) -> usize { | ^^^^^^^^^^^^^^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_without_is_empty = note: `#[warn(clippy::len_without_is_empty)]` on by default
this `let...else` may be rewritten with the `?` operator: crates/kernel/src/vm/moo_frame.rs#L274
warning: this `let...else` may be rewritten with the `?` operator --> crates/kernel/src/vm/moo_frame.rs:274:9 | 274 | / let Some(scope) = self.scope_stack.pop() else { 275 | | return None; 276 | | }; | |__________^ help: replace it with: `let scope = self.scope_stack.pop()?;` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#question_mark = note: `#[warn(clippy::question_mark)]` on by default