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.

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

refactor(publisher): Remove PublishError in favor of anyhow::Result #312

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Commits on Nov 14, 2024

  1. refactor(publisher): remove PublishError in favor of anyhow::Result

    This commit removes PublishError in favor of bubbling pending
    unactionable errors to std when there is an error publishing
    any subject/payload.
    
    Given the focus of the Publish process, only dependent libraries or
    actionable errors should have an error signature.We can safely do this
    since it's all application code and we will be relying on our tracing
    for stack traces to implement certain actions after analyzing the errors.
    This makes our code more concise for the domain solution it solves.
    Jurshsmith committed Nov 14, 2024
    Configuration menu
    Copy the full SHA
    3f5f663 View commit details
    Browse the repository at this point in the history