Skip to content

Conversation

@torkelrogstad
Copy link
Contributor

@torkelrogstad torkelrogstad commented Jan 29, 2025

The motivation for this is being able to pick up multiple different error sources that need to be propagated out from the main function. Prior to this commit, for example being unable to start the RPC server would not end in a fatal error.

@Ash-L2L Ash-L2L force-pushed the 2025-01-24-integration-tests branch from 4a343cb to 774a171 Compare January 29, 2025 15:24
@torkelrogstad torkelrogstad force-pushed the 2025-01-29-runtime branch 2 times, most recently from 67c1eba to 6d959d0 Compare January 29, 2025 18:25
@torkelrogstad torkelrogstad changed the base branch from 2025-01-24-integration-tests to master January 29, 2025 18:25
@torkelrogstad torkelrogstad force-pushed the 2025-01-29-runtime branch 2 times, most recently from fd94d09 to a5a31b7 Compare January 30, 2025 08:08
The motivation for this is being able to pick up multiple different error sources that need to be propagated out from the main function. Prior to this commit, for example being unable to start the RPC server would not end in a fatal error.
Copy link
Collaborator

@Ash-L2L Ash-L2L left a comment

Choose a reason for hiding this comment

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

LGTM otherwise!

@Ash-L2L Ash-L2L merged commit ab7335e into master Jan 30, 2025
12 checks passed
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.

3 participants