Skip to content

Remove anyhow from store-layer crates#12692

Open
vaporif wants to merge 3 commits into
rerun-io:mainfrom
vaporif:remove-anyhow
Open

Remove anyhow from store-layer crates#12692
vaporif wants to merge 3 commits into
rerun-io:mainfrom
vaporif:remove-anyhow

Conversation

@vaporif
Copy link
Copy Markdown

@vaporif vaporif commented Mar 13, 2026

Related

What

Replace anyhow with thiserror in re_query, re_chunk_store, and re_entity_db library crates.

@vaporif vaporif marked this pull request as ready for review March 13, 2026 14:44
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi! Thanks for opening this pull request.

Because this is your first time contributing to this repository, make sure you've read our Contributor Guide and Code of Conduct.

Chunk(#[from] re_chunk::ChunkError),

#[error("Failed to load data, parsing error: {0:#}")]
Codec(#[from] re_log_encoding::CodecError),
Copy link
Copy Markdown
Author

@vaporif vaporif Mar 16, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was not being used anywhere. Safe to remove or still should have stayed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant