Skip to content

✨ 新增终末地 Platform#759

Open
GuGuMur wants to merge 14 commits intoMountainDash:mainfrom
GuGuMur:feat/endfield
Open

✨ 新增终末地 Platform#759
GuGuMur wants to merge 14 commits intoMountainDash:mainfrom
GuGuMur:feat/endfield

Conversation

@GuGuMur
Copy link
Contributor

@GuGuMur GuGuMur commented Feb 21, 2026

This change is Reviewable

@netlify
Copy link

netlify bot commented Feb 21, 2026

Deploy Preview for nonebot-bison ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 3d9db6c
🔍 Latest deploy log https://app.netlify.com/projects/nonebot-bison/deploys/69990bc323894400082fb469
😎 Deploy Preview https://deploy-preview-759--nonebot-bison.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@GuGuMur GuGuMur changed the title ❇️ 新增终末地 Platform ✨ 新增终末地 Platform Feb 21, 2026
@codecov
Copy link

codecov bot commented Feb 21, 2026

Codecov Report

❌ Patch coverage is 81.60377% with 39 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.98%. Comparing base (5bf20f4) to head (3d9db6c).

Files with missing lines Patch % Lines
nonebot_bison/platform/endfield.py 81.59% 37 Missing ⚠️
nonebot_bison/platform/arknights.py 88.88% 1 Missing ⚠️
nonebot_bison/theme/themes/arknights/build.py 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #759      +/-   ##
==========================================
- Coverage   85.10%   84.98%   -0.13%     
==========================================
  Files          99      100       +1     
  Lines        5641     5842     +201     
==========================================
+ Hits         4801     4965     +164     
- Misses        840      877      +37     
Flag Coverage Δ
smoke-test 84.98% <81.60%> (-0.13%) ⬇️

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

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.



class ArknightsPost(Post, HTMLContentSupport):
class HGAnnouncePost(Post, HTMLContentSupport):
Copy link
Collaborator

Choose a reason for hiding this comment

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

需要这个HG缩写吗,直接叫AnnouncePost就可以吧,反正是特定platform下的类,不用担心有什么冲突

from .platform import NewMessage, RawPost, StatusChange


class EFResponseBase(BaseModel):
Copy link
Collaborator

Choose a reason for hiding this comment

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

  1. 没必要缩写,写就写全更能看懂
  2. 以及这个model这么多的话可以单开一个文件比较好

return CleanedBulletinData(title=title, content=content, images=images, link=link)


class EFBulletinListResponse(EFResponseBase):
Copy link
Collaborator

Choose a reason for hiding this comment

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

缩写同理

data: BulletinList


class EFBulletinResponse(EFResponseBase):
Copy link
Collaborator

Choose a reason for hiding this comment

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

缩写同理

)


class EFVersion(StatusChange):
Copy link
Collaborator

Choose a reason for hiding this comment

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

缩写同理

class ArknightsSite(Site):
name = "arknights"
class HypergryphSite(Site):
name = "hypergryph"
Copy link
Collaborator

Choose a reason for hiding this comment

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

这个name可能不能改,因为 db 里的 cookie 表写了这个名字,最好加个 db 的 migration 把原来的数据改了

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.

3 participants