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.

Sign up for GitHub

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

Jump to bottom

🐛 修复加载功能时 matcher 无法正确获取的问题 #9

Merged
merged 1 commit into from
Aug 23, 2023
Merged

🐛 修复加载功能时 matcher 无法正确获取的问题 #9

merged 1 commit into from
Aug 23, 2023

Conversation

A-kirami
Copy link
Owner

这个 PR 带来了什么样的更改？

  • 错误修复
  • 新功能
  • 文档/注释
  • 代码格式
  • 代码重构
  • 测试用例
  • 性能优化
  • 外观样式
  • 项目构建
  • 依赖环境
  • 持续集成/部署
  • 其他，请描述:

这个 PR 是否存在破坏性变更？

  • 是的，并已在 issue #___ 号中获得批准
  • 没有

描述

增加 MatcherCase 类型判断

动机和背景

使用标准写法时，由于 KiramiBot 的事件响应器返回 MatcherCase，导致 load_abilities 方法无法正确获取 matcher

其他信息

检查工作

  • 我对我的代码进行了注释，特别是在难以理解的部分
  • 我的更改需要更新文档，并且已对文档进行了相应的更改
  • 我添加了测试并且已经在本地通过，以证明我的修复补丁或新功能有效
  • 我已检查并确保更改没有与其他打开的 Pull Requests 重复

@A-kirami A-kirami added the bug Something isn't working label Aug 23, 2023
@A-kirami A-kirami merged commit 1ad2ccc into main Aug 23, 2023
3 checks passed
@A-kirami A-kirami deleted the fix/load-abilities branch August 23, 2023 23:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@A-kirami