timescaledb-toolkit icon indicating copy to clipboard operation
timescaledb-toolkit copied to clipboard

cargo update atomic-polyfill and cpufeatures

Open epgts opened this issue 2 years ago • 4 comments

These were yanked, as reported by cargo audit:

Crate: atomic-polyfill Version: 0.1.10 Warning: yanked

Crate: cpufeatures Version: 0.2.2 Warning: yanked

Crate: critical-section Version: 0.2.7 Warning: yanked

epgts avatar Nov 10 '22 19:11 epgts

oh boy:

error: use of deprecated associated function `pest::prec_climber::PrecClimber::<R>::climb`:
Use `pest::pratt_parser` instead (it is an equivalent which also supports unary prefix/suffix operators).
       While prec_climber is going to be kept in 2.x minor and patch releases,
       it may be removed in a future major release.
  --> extension/src/time_vector/pipeline/lambda/parser.rs:53:18

What do you think @WireBaron?

  • update the lambda stuff
  • remove the lambda stuff
  • see if we can get clippy to ignore these deprecations
  • see if we can update pest less hard

epgts avatar Nov 10 '22 20:11 epgts

Let's discuss this next Tuesday and see if anyone's willing to take over updating this. It might fit in with some of Thomas's work on making user defined outputs for TimeVectors, though I doubt his solution would be going down to the level of pest.

WireBaron avatar Nov 11 '22 20:11 WireBaron

@WireBaron freshened and ready to go

epgts avatar Jan 05 '23 17:01 epgts

@WireBaron I just checked and this still merges cleanly and passes tests.

epgts avatar Feb 09 '23 15:02 epgts