Skip to content

style(android): 移除已完成的TODO#299

Open
XiaoBuHaly wants to merge 1 commit intoChevey339:masterfrom
XiaoBuHaly:chore/TODO-remove-1
Open

style(android): 移除已完成的TODO#299
XiaoBuHaly wants to merge 1 commit intoChevey339:masterfrom
XiaoBuHaly:chore/TODO-remove-1

Conversation

@XiaoBuHaly
Copy link
Copy Markdown
Contributor

@XiaoBuHaly XiaoBuHaly commented Feb 1, 2026

style(android): 清理 defaultConfig TODO 注释

移除无价值 TODO 注释 · 仅注释清理 · 构建行为不变

PR 类型
Style
破坏性变更
文件数
1
代码增删
▱▱▱▱▱▱▱▱▱▱
+0 -1


核心目标

1 注释噪音 — 移除 android/app/build.gradle.ktsdefaultConfig 段的模板已完成的 TODO 注释,减少无意义提示。

审阅指引

可选审阅以下部分:

优先级 文件 原因
P0 android/app/build.gradle.kts 本次 PR 唯一改动点

验证清单

构建验证(可选)
flutter build apk

预期:构建通过;产物行为不变


本 PR 由 XiaoBuHaly:chore/TODO-remove-1 发起

@XiaoBuHaly
Copy link
Copy Markdown
Contributor Author

这个 PR 改动只有关于 TODO 的注释

@luosc
Copy link
Copy Markdown
Contributor

luosc commented Feb 26, 2026

我猜这个是给contributor留的todo吧?让他们把包名改成自己的

@XiaoBuHaly
Copy link
Copy Markdown
Contributor Author

我猜这个是给contributor留的todo吧?让他们把包名改成自己的

感谢回复!关于这个TODO注释,我想从几个角度补充一下:

  1. 注释来源
    这个TODO是Flutter模板在创建项目时自动生成的,它的本意是提醒所有者在构建或上架前设置一个唯一的Application ID。一旦项目初始化完成并设置了包名(比如这里的com.psyche.kelivo),这个提醒就已经完成了它的使命,不再具有实际价值。

  2. TODO注释的使用场景
    TODO在代码中通常用来标记待完成的工作、待修复的问题或待决策的事项,属于项目内部开发过程中的跟踪标记,而非长期保留的“提示牌”。如果这个注释仅用于提醒其他贡献者“fork后记得改包名”,那它实际上并没有标记任何未完成的工作——包名已经写好,且对原项目本身而言不需要改动。因此,它已经超出了TODO的合理用途。

  3. 更好的信息传递方式
    如果希望贡献者在fork后修改包名以避免冲突,更合适的方式是在README贡献指南中明确说明,例如“如果你计划修改并发布自己的版本,请记得更改applicationId以避免与官方包名冲突”。这样既不会污染代码,也能更清晰地传达意图。

所以,我认为这个TODO注释现在已经没有存在的必要,移除它能避免让其他开发者产生困惑(比如误以为还有未完成的配置)。当然,如果项目确实希望保留某种提示,建议用文档而非TODO来实现。

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.

2 participants