Skip to content

新增 Gradle 构建支持,与 Maven 双构建共存,并添加日语支持。#11

Closed
EndlessPixel wants to merge 4 commits into
YanPlugins:masterfrom
EndlessPixel:master
Closed

新增 Gradle 构建支持,与 Maven 双构建共存,并添加日语支持。#11
EndlessPixel wants to merge 4 commits into
YanPlugins:masterfrom
EndlessPixel:master

Conversation

@EndlessPixel
Copy link
Copy Markdown
Contributor

@EndlessPixel EndlessPixel commented Apr 26, 2026

✨ 变更内容
本 PR 为项目新增 Gradle 构建支持,实现 Maven + Gradle 双构建共存,并添加日语支持。

  • 新增 Gradle 构建配置文件,与现有 Maven 构建逻辑保持一致
  • 新增 Gradle Wrapper,无需预装 Gradle 即可编译
  • 新增 日语 语言文件
  • 优化 .gitignore,兼容 Maven & Gradle 构建产物
  • 不修改任何业务代码
  • 不影响原有 CI 流程
  • 不破坏现有功能

📌 重点说明

  • 原 Maven 构建可正常使用
  • 无破坏性变更
  • 仅新增构建与语言相关文件

@EndlessPixel EndlessPixel changed the title 新增 Gradle 构建支持,与 Maven 双构建共存 新增 Gradle 构建支持,与 Maven 双构建共存,并添加日语支持。 Apr 28, 2026
@baicaizhale
Copy link
Copy Markdown
Member

baicaizhale commented Apr 30, 2026

我没有抨击你的意思,有几句忠言你需要看看,不要让ai帮你读

今天也许换其他大开源项目的作者看见你这个PR都要当成恶作剧直接屏蔽了,但是我完全理解你的热情,建议你vibecoding的时候看一眼ai都干了什么,你的提示词很可能让ai误会了应该干什么

还有你需要学一下Git怎么用(你的commit message Co-authored-by: Copilot <copilot@github.com>,说明你的commit居然是copilot帮你打的,然后copilot也是毫不客气的写上自己的大名)
以及怎么编译一个Maven项目(你可能是在按网上教程执行gradle的构建命令后gradle告诉你没有找到gradle的配置文件,你看不懂所以你告诉ai,ai完全误会你的意思所以写了一套gradle的配置)

最后这是GPT的评价(可能说话有点狠):

总结陈词:
这位 EndlessPixel 同学展现了一个经典的开源新手画像:热情有余,细心不足,且极度缺乏测试意识。
整个 PR 像是一次"我来刷存在感"的即兴表演:给项目加个 Gradle 构建(虽然版本号写错)、给 .gitignore 做个 SPA、顺便"支持"一下日语(通过消灭中文来实现)、最后更新 README 给自己的其他项目引流。最讽刺的是,一个号称"不影响原 Maven 构建"的 PR,实际上通过 Patch 3 直接让原项目的中文支持功能物理消失了。
如果我是项目 maintainer,我会:礼貌地感谢他的热情,然后关闭 PR,建议他先学会用 git diff 检查自己提交了什么,再学会用 git rebase 把四个 commit 压成一个,最后重新提交一个只包含 Gradle 支持的干净 PR——当然,前提是先把 Gradle 版本号改对,以及把中文语言文件从日文翻译回来。

@baicaizhale baicaizhale added the invalid This doesn't seem right label Apr 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

invalid This doesn't seem right

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants