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: 事件响应器的 __repr__ 方法添加 rule 信息 #2295

Closed
wants to merge 1 commit into from

Conversation

A-kirami
Copy link
Contributor

目前 __repr__ 方法展示的信息区分度不足,同一插件内的多个事件响应器无法区分。

通过添加 rule 信息,能够很好的区分不同事件响应器。

@codecov
Copy link

codecov bot commented Aug 23, 2023

Codecov Report

Merging #2295 (f44f4ba) into master (117ef18) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2295   +/-   ##
=======================================
  Coverage   89.92%   89.92%           
=======================================
  Files          46       46           
  Lines        3345     3345           
=======================================
  Hits         3008     3008           
  Misses        337      337           
Flag Coverage Δ
unittests 89.92% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
nonebot/internal/matcher/matcher.py 97.05% <ø> (ø)

@github-actions
Copy link
Contributor

🚀 Deployed on https://deploy-preview-2295--nonebot2.netlify.app

@github-actions github-actions bot temporarily deployed to pull request August 23, 2023 23:16 Inactive
@yanyongyu
Copy link
Member

之前就是你们觉得日志太长才把这些东西删掉的

@yanyongyu
Copy link
Member

rule信息展示出来会很长,可以考虑展示 lineno

@A-kirami
Copy link
Contributor Author

rule信息展示出来会很长,可以考虑展示 lineno

或许可以考虑只在 debug 日志中展示详细信息?

@yanyongyu
Copy link
Member

当初就是说的debug日志太长了啥都要找

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants