Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

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

Merged
merged 2 commits into from
Sep 24, 2024
Merged

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

merged 2 commits into from
Sep 24, 2024

Conversation

ethanoroshiba
Copy link
Contributor

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
42 checks passed
@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
Reviewers

@SuperFluffy SuperFluffy SuperFluffy approved these changes

Assignees
No one assigned
Labels
sequencer pertaining to the astria-sequencer crate
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Sequencer is not installing astria-eyre error hook
2 participants
@ethanoroshiba @SuperFluffy