actually this doesn't work.

see also https://doc.rust-lang.org/nightly/cargo/reference/manifest.html#the-rust-version-field but sadly this is nightly

Codecov Report

Merging #724 (d64b62c) into master (deb31a0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #724   +/-   ##
=======================================
  Coverage   82.51%   82.51%           
=======================================
  Files         168      168           
  Lines       47647    47647           
=======================================
  Hits        39317    39317           
  Misses       8330     8330           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update deb31a0...d64b62c. Read the comment docs.

I feel like we could still merge as is and wait for the stable toolchain to catch up evetually.

I feel like we could still merge as is and wait for the stable toolchain to catch up evetually.

sadly this option is ready in 1.56 by which time rust edition 2021 is already out

I feel like we could still merge as is and wait for the stable toolchain to catch up evetually.

sadly this option is ready in 1.56 by which time rust edition 2021 is already out

For now it only displays a warning, no?

An alternate is to document that arrow rs only tests on the latest rust stable: #726

If the goal is to ensure that the arrow and parquet crates work on Rust 1.53 I suggest we add a CI run that at least cargo checks that they can be compiled with 1.53.