pin comfy-table to 7.1.2

[zachschuermann]

Which issue does this PR close?

• Closes comfy-table release 7.2.0 breaks MSRV #8243 .

What changes are included in this PR?

pin comfy-table to release prior to 7.2.0's MSRV bump to 1.85 - included a TODO to unpin after arrow bumps to 1.85

(context FWIW: caught in delta_kernel MSRV CI)

Are these changes tested?

validated MSRV with cargo-msrv:

# now passes
cargo msrv --path arrow-cast/ verify --rust-version 1.84 --all-features

[mbrobbel]

Thanks @zachschuermann

[scovich]

Would it make sense to ship a point release for this? The current release is unusable because of contradictory MSRV.
To resolve the contradiction, either the MSRV needs to bump or this fix here needs to ship.

[mbrobbel]

Would it make sense to ship a point release for this? The current release is unusable because of contradictory MSRV. To resolve the contradiction, either the MSRV needs to bump or this fix here needs to ship.

I don't think it really matters. For downstream users without specific MSRV requirements there is no problem, and users with the same MSRV requirements can just pin the version - like we did here. The benefit of a release is that we prevent the impacted users from having to manually pin and/or to avoid an unintended bump of this dependency.

[scovich]

users with the same MSRV requirements can just pin the version - like we did here. The benefit of a release is that we prevent the impacted users from having to manually pin and/or to avoid an unintended bump of this dependency.

Confirmed that delta-kernel-rs was able to pin the version by manually adding it as a dependency (even tho it's not a direct dependency). Annoying but worked.