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

Better action handling error logs printing #821

Closed
wants to merge 2 commits into from

Conversation

varsill
Copy link
Contributor

@varsill varsill commented Jun 10, 2024

This PR:

  • Uses Membrane.Logger.error to print the original error while handling action returned by callback.
  • Improves error description formatting.

@varsill varsill requested a review from mat-hek as a code owner June 10, 2024 12:26
@varsill varsill added the no-changelog This label has to be added if changes from the PR are not meant to be placed in the CHANGELOG.md label Jun 10, 2024
@varsill varsill requested review from FelonEkonom June 10, 2024 12:31
@varsill
Copy link
Contributor Author

varsill commented Jun 10, 2024

closing in favour of: #822

@varsill varsill closed this Jun 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog This label has to be added if changes from the PR are not meant to be placed in the CHANGELOG.md
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant