Repository navigation
feat!: 同步原野之风三模组及已完成功能,完善升级文档与 CI - #34
cuccuc377-dev wants to merge 10 commits into
Conversation
Integrate WildWindMod main at eb12f4f into the upstream NeoForge 26.2 line. Deliver Core, Wilderness and Barren as separate mods with existing features and tests. Document namespace migration limits and provenance, align metadata with existing licenses, and harden CI with pinned actions, read-only permissions and three-JAR validation.
There was a problem hiding this comment.
八百多个 json 文件传上来干什么,不是有 datagen 了吗?hfas 模块也有例子
There was a problem hiding this comment.
Component.literal 加中文?为什么不用 Component.translatable?
There was a problem hiding this comment.
这个 wildwind$color 莫名其妙,为什么不用 dyeColor.textColor?
|
问题太多,都标进去了,主要是都能 datagen 为什么上传结果文件,中文为什么要硬编码进代码,还有一堆莫名其妙的代码,AI 写完都不知道你看没看过。这个 CR 显然不能过,代码质量太差 |
|
感谢审核。重新逐项核对了意见和实现,中文硬编码、重复维护颜色表,以及生成资源的组织方式确实需要改进。此前构建和自动测试通过,只能说明已覆盖的行为通过验证,不能替代国际化、可维护性和仓库约定方面的审查。 针对几处实现补充说明:
这些是对当前实现的说明和拟修订方向,尚未提交修复。会先处理上述问题,再申请复审,不以 CI 通过作为绕过这些意见的理由。 |
问题与改动
当前默认分支仍是单个
ww_hfas模组,尚未包含 WildWindMod 中完成的功能和三模组拆分。本 PR 将cuccuc377-dev/WildWindMod的main@eb12f4fed5472c9be5c51606141fff201b873968同步到master/NeoForge-26.2,发行目录只产出以下三个独立 JAR:wild_wind_corewild_wind_wildernesswild_wind_barren包含直接署名的属性书、附魔台可获取的熔炼魔咒与保持贴图原色的树叶。按需求移除未完成的自定义部件制箭系统,保留原版制箭台行为与已有延展药水箭。
补充同步/升级记录、贡献指南及 PR 模板;CI 固定 Actions 提交、令牌只读、限制超时、取消重复运行、上传测试报告并验证恰好三个发行模组。JAR 许可标识与现有仓库双许可声明保持一致,许可证原文未改变。
来源与历史
85926800c060020dd9afbf3c4a8ab90a2540421c。26ad2da288d7018a1eb39d5bfbf30b2ad0a1520c;目标基线是该提交的祖先,目标现有泥沼蟹等改动已包含。docs/PROVENANCE.md、docs/UPSTREAM_MERGE.md、docs/UPSTREAM_SYNC.md。26ad2da多 8 个提交,涉及木材生成器、标签/配方和恐鱼音效等;本 PR 不声称包含这些额外提交。请协调合并顺序并重新验证重叠部分。兼容性
此同步包含注册命名空间破坏性变化,没有实现旧
ww_hfas:*存档自动迁移。 先备份并在新测试世界验证;旧世界、结构 NBT 和数据包中的旧 ID 需要按模块转换。恐鱼完整 ID 为wild_wind_wilderness:piranha。移除旧
ww_hfas/独立ww_structuresJAR,再安装需要的模块;客户端和服务端组合一致。荒野还需要 Minecraft 26.2 NeoForge 版 GeckoLib 5.5.4。ww_structures配置路径、结构命名空间和索引保留;删除的自定义制箭台不自动恢复。回退应恢复备份世界及对应旧 JAR。验证
在独立同步工作区、JDK 25 下完成:
:datagen:runClientData :datagen:runServerData,同步 417 项生成资源后快照无差异。build collectMods :verification:runGameTest。:smoke:runGameTest四种组合通过:核心、核心+荒野、核心+瘠土、全部模块。git diff --check;提交树中排除 IDE、工具、凭据、构建缓存、测试世界和发行产物,保留构建所需 Wrapper。GitHub Actions 已独立重复资源生成、完整构建、综合测试与安装组合验证,两次运行均通过:
CI 产物
WildWind-mc26.2包含三个发行 JAR,WildWind-test-reports保留测试报告。未进行本次同步版本的长期游玩、完整人工视觉或多人验收。目标分支要求至少一位审核者批准。本 PR 等待 CI 与正常审核,不绕过分支保护。