Skip to content

重构 GameRepository - #6580

Open
Glavo wants to merge 212 commits into
HMCL-dev:mainfrom
Glavo:game-repo
Open

重构 GameRepository#6580
Glavo wants to merge 212 commits into
HMCL-dev:mainfrom
Glavo:game-repo

Conversation

@Glavo

@Glavo Glavo commented Aug 2, 2026

Copy link
Copy Markdown
Member

当前 GameRepository 的职责很混乱,存在大量线程安全性问题。本 PR 尝试重构 GameRepository,解决它所存在的问题。

@Glavo
Glavo marked this pull request as draft August 2, 2026 12:51
@github-actions github-actions Bot added the 100+ label Aug 2, 2026
Glavo added 16 commits August 2, 2026 20:54
…y layout defaults

Assisted-by: grok-build:grok-4.5
…aultGameRepositoryLayout to abstract classes for improved extensibility
@github-actions github-actions Bot added 2000+ and removed 100+ labels Aug 3, 2026
Glavo added 22 commits August 15, 2026 21:23
@Glavo
Glavo marked this pull request as ready for review August 19, 2026 12:49
@Glavo

Glavo commented Aug 19, 2026

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Swish!

Reviewed commit: e559bbff7e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@Glavo

Glavo commented Aug 19, 2026

Copy link
Copy Markdown
Member Author

虽然这个 PR 可能还有很多问题,不过因为这是很重要的大型重构,我考虑近期把它合并,然后再慢慢解决这些问题。

考虑到 3.16 到现在没有很多重要更新,我准备将 3.17 推迟到下个月发布,用更多时间完成基础重构。

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant