-
Notifications
You must be signed in to change notification settings - Fork 20
feat(bilibili): 增强哔哩哔哩图文动态解析&修复字体占用 #444
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
base: master
Are you sure you want to change the base?
Conversation
- 支持解析哔哩哔哩图文动态内容,并将动态和图文信息合并为统 一接口。同时修复了代码格式化问题,优化了部分函数参数 换行以提高可读性。 - Bug: 字体资源被持续占用未释放 Fixes fllesser#433 - Bug: B站动态解析缺少图片资源 Fixes fllesser#443 - Bug: 动态解析错误 Fixes fllesser#442
Codecov Report❌ Patch coverage is
|
移除重复的opus测试函数,将测试数据合并到dynamic测试中, 统一测试逻辑并优化代码结构,提高测试效率。
移除了对动态解析结果中标题字段的非空断言, 保留了作者字段的断言以确保基本功能正常
| @handle("/BV", r"bilibili\.com(?:/video)?/(?P<bvid>BV[0-9a-zA-Z]{10})(?:\?p=(?P<page_num>\d{1,3}))?") | ||
| @handle( | ||
| "/BV", | ||
| r"bilibili\.com(?:/video)?/(?P<bvid>BV[0-9a-zA-Z]{10})(?:\?p=(?P<page_num>\d{1,3}))?", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里为什么要换行
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
格式化插件自动的
| @handle("/av", r"bilibili\.com(?:/video)?/av(?P<avid>\d{6,})(?:\?p=(?P<page_num>\d{1,3}))?") | ||
| @handle( | ||
| "/av", | ||
| r"bilibili\.com(?:/video)?/av(?P<avid>\d{6,})(?:\?p=(?P<page_num>\d{1,3}))?", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
同
一接口。同时修复了代码格式化问题,优化了部分函数参数 换行以提高可读性。
Fixes Bug: 字体资源被持续占用未释放 #433
Fixes Bug: B站动态解析缺少图片资源 #443
Fixes Bug: 动态解析错误 #442