Skip to content

Abstract the meow function#334

Merged
pvdz merged 3 commits intomainfrom
absmeow
Feb 21, 2025
Merged

Abstract the meow function#334
pvdz merged 3 commits intomainfrom
absmeow

Conversation

@pvdz
Copy link
Contributor

@pvdz pvdz commented Feb 20, 2025

(This depends on #333 because Sentry)

Abstract the meowOrExit function. This allows us to have a central place to hook into the meows. Case in point: add a tag to Sentry for which command was called to help debugging.

Next PR will add another reasons :d

This PR also cleans up and simplifies the API around calling meow and help etc.

Copy link
Contributor

@jdalton jdalton left a comment

Choose a reason for hiding this comment

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

I trust your process

@pvdz pvdz merged commit d53f3d2 into main Feb 21, 2025
15 checks passed
@pvdz pvdz deleted the absmeow branch February 21, 2025 15:27
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