update various dependencies

[decathorpe]

This is a followup for #938.

• update dirs from v4 to v5 (no code changes)
• update term_grid from v0.1 to v0.2 (small code changes)
• update terminal_size from v0.1 to v0.3 (no code changes)
• update lscolors from v0.15 to v0.16 (no code changes)
• update serde_yaml from v0.8 to v0.9 (no code changes)

The term_grid update required small changes to the use of term_grid::Cell because with v0.2, it is possible to specify the alignment within the cell. Adding Alignment::Left should preserve the current behaviour.

[muniu-bot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: decathorpe

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (73eeb7f) 84.44% compared to head (398bf0c) 85.08%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #984      +/-   ##
==========================================
+ Coverage   84.44%   85.08%   +0.63%     
==========================================
  Files          51       51              
  Lines        5067     5035      -32     
==========================================
+ Hits         4279     4284       +5     
+ Misses        788      751      -37     

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

[zwpaper]

Hi @decathorpe, thanks for the update, as you can see CI, once we upgrade the dirs dep to v5, the Windows build will fail.

can you help to fix it?

[decathorpe]

I briefly looked at the failure, and it seems to be a toolchain issue? I have no idea why updating Rust code should cause linker failures.

If it's dirs v5 that causes the issue, I can drop that one from the batch and we can look into the failure separately.

[zwpaper]

that would also be okay to leave dirs in a separate PR

[decathorpe]

I've dropped the dirs v4 → v5 update for now.

One thing that looks suspicious is that with v5, it now uses windows-sys instead of winapi ...

[decathorpe]

Strange. I dropped the dirs update, and the windows-gnu builds still failed in the same way.

[decathorpe]

I tried something else, maybe it changes things, maybe not. 🤞🏼

[decathorpe]

Looks like good ol' cargo update fixed it. Judging by the previous contents of Cargo.lock, it hadn't been done in years, so it's not surprising that it pinned some crates to versions that were missing fixes.

(After cargo update I made small adjustments to keep the MSRV at 1.70 - clap v4.5 bumped its MSRV to 1.74, so I kept it at v4.4.)

[zwpaper]

thanks so much for the update!

[decathorpe]

Great, thanks!