Skip to content

chore: move ApplyHandler to slogm and check handler is enabled#12

Merged
Semior001 merged 8 commits intocappuccinotm:masterfrom
ArtARTs36:master
Feb 15, 2026
Merged

chore: move ApplyHandler to slogm and check handler is enabled#12
Semior001 merged 8 commits intocappuccinotm:masterfrom
ArtARTs36:master

Conversation

@ArtARTs36
Copy link
Copy Markdown
Contributor

No description provided.

@ArtARTs36 ArtARTs36 requested a review from Semior001 as a code owner February 27, 2025 21:01
Comment thread slogm/handler.go Outdated
Copy link
Copy Markdown
Member

@Semior001 Semior001 left a comment

Choose a reason for hiding this comment

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

Only one minor thing remains, sorry for a long wait

@Semior001
Copy link
Copy Markdown
Member

It appears that intermediate handler stops the chain in case if it raises an error. I've added an option to log them in case they're needed, but the default behavior will be just to ignore them.

@Semior001 Semior001 merged commit 0ed7588 into cappuccinotm:master Feb 15, 2026
1 check 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.

2 participants