feat(deps): replace simple-tqdm with indicatif 0.18 to avoid unmaintained number_prefix

[fgimenez]

fixes errors like this one in https://github.com/tempoxyz/tempo/actions/runs/19462129274/job/55688586205?pr=948:

cargo deny --all-features check all
  shell: /usr/bin/bash -e {0}
  env:
    CARGO_TERM_COLOR: always
    CARGO_HOME: /home/runner/.cargo
    CARGO_INCREMENTAL: 0
error[unmaintained]: number_prefix crate is unmaintained
    ┌─ /home/runner/work/tempo/tempo/Cargo.lock:487:1
    │
487 │ number_prefix 0.4.0 registry+https://github.com/rust-lang/crates.io-index
    │ ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ unmaintained advisory detected
    │
    ├ ID: RUSTSEC-2025-0119
    ├ Advisory: https://rustsec.org/advisories/RUSTSEC-2025-0119
    ├ The `number_prefix` crate is no longer actively maintained.  If you rely on this crate, consider switching to a recommended alternative.
      
      ## Recommended alternatives
      
      - [`unit-prefix`](https://crates.io/crates/unit-prefix)
    ├ Announcement: https://github.com/ogham/rust-number-prefix/pull/8
    ├ Solution: No safe upgrade is available!
    ├ number_prefix v0.4.0
      └── indicatif v0.17.11
          └── simple-tqdm v0.2.0
              ├── tempo-bench v0.4.0
              └── tempo-xtask v0.4.0

[SuperFluffy]

Bump indicatif instead. They have migrated to unit-prefix: https://github.com/console-rs/indicatif/blob/2f2b2ccbd3695e0a3cdbfed42c0d60190f732bbc/Cargo.toml#L18

[fgimenez]

Bump indicatif instead. They have migrated to unit-prefix: https://github.com/console-rs/indicatif/blob/2f2b2ccbd3695e0a3cdbfed42c0d60190f732bbc/Cargo.toml#L18

@SuperFluffy yeah, the problem is that simple-tqdm is locked to indicatif@0.17.7 and this one pulls the unmaintained crate, no new version of simple-tqdm atm. we could try to update the code to use indicatif directly instead of simple-tqdm, can do that in a followup, wdyt?

[SuperFluffy]

Bump indicatif instead. They have migrated to unit-prefix: https://github.com/console-rs/indicatif/blob/2f2b2ccbd3695e0a3cdbfed42c0d60190f732bbc/Cargo.toml#L18

@SuperFluffy yeah, the problem is that simple-tqdm is locked to indicatif@0.17.7 and this one pulls the unmaintained crate, no new version of simple-tqdm atm. we could try to update the code to use indicatif directly instead of simple-tqdm, can do that in a followup, wdyt?

We should absolutely migrate. Looking at simple-tqdm, it was never maintained in the first place.

IMO we should either do it now or close this PR. We shouldn't even start silencing nags like this one if there is a clear fix (lest we forget to actually followup).

[fgimenez]

We should absolutely migrate. Looking at simple-tqdm, it was never maintained in the first place.

makes sense, done in this PR ptal

[SuperFluffy]

looks good - we got cloudflare issues?

[SuperFluffy]

This has one confusing commit message now. :-D

[fgimenez]

This has one confusing commit message now. :-D

hmm this is weird i changed the PR title and amended the commit message...

[SuperFluffy]

This has one confusing commit message now. :-D

hmm this is weird i changed the PR title and amended the commit message...

It's because you added the PR to the merge queue before your changes. The only "sync point" between PR title/body and commit title/body is when you enable auto-merge - that's the moment it takes the PR title/body and suggests it for your commit.

It happened to me as well before. :-)

[fgimenez]

It's because you added the PR to the merge queue before your changes. The only "sync point" between PR title/body and commit title/body is when you enable auto-merge - that's the moment it takes the PR title/body and suggests it for your commit.

TIL, very good to know thx!