Skip to content

chore: fix some typos#333

Open
shuangcui wants to merge 40 commits intoLight-City:masterfrom
shuangcui:master
Open

chore: fix some typos#333
shuangcui wants to merge 40 commits intoLight-City:masterfrom
shuangcui:master

Conversation

@shuangcui
Copy link
Copy Markdown

No description provided.

@Light-City
Copy link
Copy Markdown
Owner

感谢你的贡献!

这个 PR 创建于 2024 年,已经有一段时间了。我注意到这个 PR 虽然标题是修复 typo,但实际上添加了很多 Bazel 构建文件和 VS Code 配置。

问题分析:

  1. 范围过大:PR 修改了 25+ 个文件,包括添加构建系统配置
  2. 有冲突:由于仓库已经更新,这个 PR 现在有冲突需要解决
  3. 关注点分离:Typo 修复和构建系统配置应该是分开的 PR

建议:

  1. 如果主要是修复 typo,请创建一个新的 PR 只包含 typo 修复
  2. 如果需要添加 Bazel 构建支持,请创建一个专门的 PR 并说明为什么需要
  3. 考虑将大 PR 拆分成多个小 PR,每个关注一个特定方面

由于这个 PR 已经过时且有冲突,我将关闭它。期待看到你新的、更专注的贡献!

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.

8 participants