Update dependencies which do not bump minimum rustc

[shssoichiro]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.54%. Comparing base (3c3a26f) to head (025a405).
Report is 11 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3359      +/-   ##
==========================================
+ Coverage   88.52%   88.54%   +0.01%     
==========================================
  Files          89       89              
  Lines       28323    28355      +32     
==========================================
+ Hits        25074    25107      +33     
+ Misses       3249     3248       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[FreezyLemon]

This breaks the WASM build. Bumpalo 3.15.0 changed the MSRV to 1.73.

[shssoichiro]

I would be really happy if more Rust maintainers would consider a compiler version change as breaking... anyway I guess I'll add a CI step for wasm, looks like we're missing them.

[FreezyLemon]

MSRV bump being considered semver breaking or not seems to be a very contentious issue..
It would be a lot better if cargo update honored the MSRV, I think a lot of issues would be inherently solved by that

[FreezyLemon]

BTW I recommend using cargo-hack to validate MSRV (and other things):
cargo +1.70 hack --each-feature check

Might be worth adding something to CI, although a job like that would take very long to complete...

[shssoichiro]

We do have CI jobs for validating minimum rust version. Just not one for wasm currently. hack might be useful for local testing before pushing up to CI.

[shssoichiro]

It looks like adding wasm tests in CI here may be a bit more complicated than in v_frame. But I'm also not an expert at Github actions. Anyway, for now I'll create an issue to follow up on adding wasm tests to CI, so we don't forget it.

[FreezyLemon]

Everything seems to build fine on 1.70 now.

[FreezyLemon]

On second thought, maybe add a comment in the Cargo.toml that clap is held back for MSRV reasons?

[lu-zero]

It should be freshened and landed... Somehow the notification got lost.

[lu-zero]

Meh, let's bump to 1.74 and be done with that :)