CI Build

Overhaul the text parsers, port from `nom` to `winnow` (#892) #3568

Triggered via push January 9, 2025 18:02

zslayton

pushed 0943766

main

Status Failure

Total duration 6m 55s

Artifacts –

rust.yml

on: push

Matrix: Setup Build Matrix

Matrix: Build and Test

Build and Test Confirmation

Annotations

8 errors and 86 warnings

this `match` expression can be replaced with `?`: src/lazy/expanded/mod.rs#L667

error: this `match` expression can be replaced with `?` --> src/lazy/expanded/mod.rs:667:38 | 667 | let next_value = match evaluator.next() { | ______________________________________^ 668 | | Ok(value) => value, 669 | | Err(e) => return Err(e), 670 | | }; | |_____________________^ help: try instead: `evaluator.next()?` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#question_mark

this `match` expression can be replaced with `?`: src/lazy/expanded/mod.rs#L642

error: this `match` expression can be replaced with `?` --> src/lazy/expanded/mod.rs:642:42 | 642 | let resolved_e_exp = match e_exp.resolve(context_ref) { | __________________________________________^ 643 | | Ok(resolved) => resolved, 644 | | Err(e) => return Err(e), 645 | | }; | |_____________________^ help: try instead: `e_exp.resolve(context_ref)?` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#question_mark

this `match` expression can be replaced with `?`: src/lazy/expanded/mod.rs#L561

error: this `match` expression can be replaced with `?` --> src/lazy/expanded/mod.rs:561:38 | 561 | let resolved_e_exp = match e_exp.resolve(context_ref) { | ______________________________________^ 562 | | Ok(resolved) => resolved, 563 | | Err(e) => return Err(e), 564 | | }; | |_________________^ help: try instead: `e_exp.resolve(context_ref)?` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#question_mark

this `match` expression can be replaced with `?`: src/lazy/expanded/macro_evaluator.rs#L992

error: this `match` expression can be replaced with `?` --> src/lazy/expanded/macro_evaluator.rs:992:24 | 992 | let step = match current_expansion.next_step() { | ________________________^ 993 | | Ok(step) => step, 994 | | Err(e) => return Err(e), 995 | | }; | |_____________^ help: try instead: `current_expansion.next_step()?` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#question_mark

this `match` expression can be replaced with `?`: src/lazy/expanded/macro_evaluator.rs#L946

error: this `match` expression can be replaced with `?` --> src/lazy/expanded/macro_evaluator.rs:946:25 | 946 | let expansion = match macro_expr.expand() { | _________________________^ 947 | | Ok(expansion) => expansion, 948 | | Err(e) => return Err(e), 949 | | }; | |_________^ help: try instead: `macro_expr.expand()?` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#question_mark = note: `-D clippy::question-mark` implied by `-D warnings` = help: to override `-D warnings` add `#[allow(clippy::question_mark)]`

Build and Test (ubuntu, all)

could not compile `ion-rs` (lib) due to 5 previous errors

Build and Test (ubuntu, all)

Clippy had exited with the 101 exit code

Build and Test Confirmation

Process completed with exit code 1.

Setup Build Matrix (true, ion-rust, windows, macos, ubuntu, al2-x86, al2-arm, windows, macos, ubu...

ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636

Build and Test (macos, default)

The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Build and Test (macos, default)