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

feat(discord): Optimize segment.image sending #282

Merged
merged 9 commits into from
Jun 23, 2021
Merged

Conversation

dragon-fish
Copy link
Member

#281 被我搞出了毛病,所以我重开了

@dragon-fish dragon-fish requested review from XxLittleCxX and shigma May 29, 2021 17:22
@dragon-fish dragon-fish changed the base branch from master to develop May 29, 2021 17:37
@codecov
Copy link

codecov bot commented Jun 13, 2021

Codecov Report

Merging #282 (adc3742) into develop (ad25f8c) will increase coverage by 0.16%.
The diff coverage is 98.30%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #282      +/-   ##
===========================================
+ Coverage    92.36%   92.52%   +0.16%     
===========================================
  Files           57       58       +1     
  Lines        10881    11255     +374     
  Branches      2233     2323      +90     
===========================================
+ Hits         10050    10414     +364     
- Misses         831      841      +10     
Impacted Files Coverage Δ
packages/koishi-test-utils/src/app.ts 100.00% <ø> (+0.96%) ⬆️
packages/plugin-github/src/index.ts 95.65% <95.00%> (-0.12%) ⬇️
packages/koishi-core/src/adapter.ts 78.32% <100.00%> (+0.31%) ⬆️
packages/koishi-core/src/app.ts 97.72% <100.00%> (+4.59%) ⬆️
packages/koishi-core/src/context.ts 93.41% <100.00%> (-0.75%) ⬇️
packages/plugin-common/src/admin.ts 87.15% <100.00%> (+1.37%) ⬆️
packages/plugin-common/src/basic.ts 98.90% <100.00%> (+<0.01%) ⬆️
packages/plugin-common/src/handler.ts 100.00% <100.00%> (ø)
packages/plugin-eval/src/index.ts 93.10% <100.00%> (-0.34%) ⬇️
packages/plugin-eval/src/loaders/default.ts 70.21% <0.00%> (-29.79%) ⬇️
... and 14 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f7d9dca...adc3742. Read the comment docs.

@dragon-fish dragon-fish linked an issue Jun 22, 2021 that may be closed by this pull request
packages/adapter-discord/src/bot.ts Outdated Show resolved Hide resolved
packages/adapter-discord/src/bot.ts Outdated Show resolved Hide resolved
packages/adapter-discord/src/bot.ts Outdated Show resolved Hide resolved
packages/adapter-discord/src/bot.ts Outdated Show resolved Hide resolved
packages/adapter-discord/src/bot.ts Outdated Show resolved Hide resolved
@dragon-fish dragon-fish requested a review from shigma June 22, 2021 15:01
packages/adapter-discord/src/bot.ts Outdated Show resolved Hide resolved
packages/adapter-discord/src/index.ts Outdated Show resolved Hide resolved
dragon-fish and others added 2 commits June 23, 2021 21:49
话说这里是不是掉了一个 header
@shigma shigma merged commit 12207bd into develop Jun 23, 2021
@shigma shigma deleted the patch-df-20210530 branch June 23, 2021 17:39
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.

对于 adapter-discord 发送网络图片的简化建议
2 participants