Skip to content

fix(sequencer): install astria-eyre hook#1552

Merged
ethanoroshiba merged 2 commits intomainfrom
ENG-839/sequencer_eyre_fix
Sep 24, 2024
Merged

fix(sequencer): install astria-eyre hook#1552
ethanoroshiba merged 2 commits intomainfrom
ENG-839/sequencer_eyre_fix

Conversation

@ethanoroshiba
Copy link
Copy Markdown

Summary

Added astria_eyre::install() call to main.rs.

Background

#1387 switched the sequencer from anyhow to eyre usage, but did not install the astria-eyre error hook.

Changes

Added astria_eyre::install() call to main.rs.

Testing

Passing all tests

Related Issues

closes #1551

@github-actions github-actions bot added the sequencer pertaining to the astria-sequencer crate label Sep 23, 2024
@ethanoroshiba ethanoroshiba marked this pull request as ready for review September 23, 2024 20:30
@ethanoroshiba ethanoroshiba requested a review from a team as a code owner September 23, 2024 20:30
@ethanoroshiba ethanoroshiba added this pull request to the merge queue Sep 24, 2024
Merged via the queue into main with commit c1cc5e5 Sep 24, 2024
@ethanoroshiba ethanoroshiba deleted the ENG-839/sequencer_eyre_fix branch September 24, 2024 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sequencer pertaining to the astria-sequencer crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sequencer is not installing astria-eyre error hook

2 participants