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

Feature: add context method to the failed result exception #7

Merged
merged 2 commits into from
Apr 5, 2024

Conversation

lindyhopchris
Copy link
Member

Added the context method to the failed result exception, by implementing the context provider interface. In Laravel applications, this means the context will be logged with the exception.

To enable this change, the two log context helper classes had to be moved to the toolkit's loggable namespace. This is ok as a change, because they are utility helper classes rather than anything specific to a log implementation. This however makes it a breaking change.

Also moved the two log context helper classes to the toolkit's
loggable namespace.
@lindyhopchris lindyhopchris self-assigned this Apr 4, 2024
@lindyhopchris lindyhopchris merged commit 7dbdf7b into next Apr 5, 2024
4 checks passed
@lindyhopchris lindyhopchris deleted the feature/failed-result-exception-context branch April 5, 2024 19:02
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.

1 participant