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

Add error handling to messageReaction function #33

Merged
merged 1 commit into from
Jun 12, 2024

Conversation

k-taro56
Copy link
Contributor

This pull request fixes an issue in the messageReaction function by adding error handling. Previously, the function did not handle errors properly, which could lead to unexpected behavior. With this fix, any errors that occur during the execution of the function will be properly handled, improving the overall reliability of the code.

@k-taro56 k-taro56 enabled auto-merge June 11, 2024 19:16
@k-taro56 k-taro56 self-assigned this Jun 11, 2024
@k-taro56 k-taro56 requested review from soleil-colza and TBSten June 11, 2024 19:16
Copy link

codecov bot commented Jun 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (60e3e6a) to head (443c91a).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #33   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines          140       144    +4     
  Branches        19        20    +1     
=========================================
+ Hits           140       144    +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@soleil-colza soleil-colza left a comment

Choose a reason for hiding this comment

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

LGTMeow

@k-taro56 k-taro56 merged commit b9b72a7 into main Jun 12, 2024
8 checks passed
@k-taro56 k-taro56 deleted the fix/handle-over-20-reactions branch June 12, 2024 04:54
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