Skip to content

Freshen some dependencies and avoid warnings #65

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sampsyo
Copy link

@sampsyo sampsyo commented Jul 31, 2025

Freshening the time dependency avoids a Rust compiler error on 1.86 or greater. I also avoided some warning due to the tarpaulin attributes by (a) switching to the new recommended syntax for ignoring code coverage, and (b) following the compiler's (and documentation's) advice to suppress the resulting warning from that new syntax.

sampsyo added 2 commits July 31, 2025 11:48
Resolves this error when compiling with Rust 1.86.0:

error[E0282]: type annotations needed for `Box<_>`
  --> [...]/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/time-0.3.30/src/format_description/parse/mod.rs:83:9
   |
83 |     let items = format_items
   |         ^^^^^
...
86 |     Ok(items.into())
   |              ---- type must be known at this point
   |
   = note: this is an inference error on crate `time` caused by an API change in Rust 1.80.0; update `time` to version `>=0.3.35` by calling `cargo update`
It looks like the tarpaulin code coverage tool changed its
recommendation for which attribute to use to exclude code from coverage
tracking. Unfortunately, on stable Rust, this still requires excluding a
lint rule:
xd009642/tarpaulin#487
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant