influxdata / influxdata/influxdb
Remove snafu from dependencies in favor of thiserror
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 31.7k
- Forks
- 3.7k
- Avg merge
- 13h 37m
- Merged PRs (30d)
- 8
Description
> Would be nice if we could factor out `snafu` since we use `thiserror` everywhere else, but that can be done later.
_Originally posted by @hiltontj in https://github.com/influxdata/influxdb/pull/25737#discussion_r1902241405_
As part of #25737 we introduced a dependency as part of moving parts of clap_blocks into the influxdb3_clap_blocks crate. This was just to move what we needed in without too many changes. We should change over to this thiserror and remove our dependency on snafu
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing the changes from #25737 and the influxdb3_clap_blocks crate to locate the introduced snafu dependency and its uses. Replace those uses with thiserror, remove snafu from the crate dependencies, and run the crate's relevant tests to verify the migration.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 52/100