Pierre Zemb
Pierre Zemb
Hi! If there's no objection, I would like to give it a try as my first ~~PR~~ contribution to Upspin :smiley:
Sorry for my misuse of the PR term, I updated my comment.
Hi @jiazhai! How did you plan to develop this feature? How can I help? :smile:
FYI: I just pushed a PR (#76) to support time conversion. You can use any subformat defined in [Astropy.Time](https://astropy.readthedocs.io/en/v0.3/time/index.html) :smile:
Are you working on this feature @Daniel-B-Smith? I just pushed a PR for 6.3, and I was thinking about doing 7.X next.
Okay, I will try to open a 7.X PR once [a pre-release is tagged](https://forums.foundationdb.org/t/large-range-scans-avoid-5s-limit/292/31?u=pierrez).
Initial work around api 700 can be found [here](https://github.com/PierreZ/foundationdb-rs/tree/dev/700)
Mmmh, I can see some errors related to [clap](https://github.com/PierreZ/foundationdb-rs/runs/3336800152?check_suite_focus=true#step:5:152). [Clap's MSRV for >=3.0 is 1.54.0](https://github.com/clap-rs/clap#minimum-supported-version-of-rust-msrv), should we bump? Also, we are targetting `structopt = "0.3.3"` which does not exists, latest...
CI is [green on my side](https://github.com/PierreZ/foundationdb-rs/actions/runs/1277167912), but I had to bump the MSRV to fix it, `clang-sys`'s transitives dependencies requires 1.46.
Thanks for the early review @Speedy37 :+1: