Skip to content

Artifacts 官中拉取并修改#2228

Merged
Cactusstudent merged 1 commit intoCFPAOrg:mainfrom
RCqaq:artifacts
Nov 15, 2022
Merged

Artifacts 官中拉取并修改#2228
Cactusstudent merged 1 commit intoCFPAOrg:mainfrom
RCqaq:artifacts

Conversation

@RCqaq
Copy link
Copy Markdown
Contributor

@RCqaq RCqaq commented Jul 18, 2022

官中加戏太多,且有不少错误。

  • 我已仔细阅读贡献指南 CONTRIBUTING
  • 我已确认标题符合{模组英文全名} {简述}的格式,如Ender IO 翻译新增/更新/修正/删除
  • 我已确认英文原文(如 en_us.json)存在且完整,内容与中文对应;
  • 我已确认提交文件的路径名称正确例子);
    • 如果是 1.12 翻译,应该是:projects/1.12.2/assets/{CurseForge 项目名称}/{ModID}/lang/zh_cn.lang
    • 如果是 1.16 及以上的翻译,应该是:projects/{版本}/assets/{CurseForge 项目名称}/{ModID}/lang/zh_cn.json
  • 我已阅读并同意按 CC BY-NC-SA 4.0 协议发布我的作品;
  • 刷新 PR 的标签/状态,有需要再点击;

@boring-cyborg boring-cyborg Bot added 1.16.x Minecraft 版本标签 1.18.x Minecraft 版本标签 Forge 包含 Forge 模组 labels Jul 18, 2022
@cfpa-bot
Copy link
Copy Markdown
Contributor

cfpa-bot Bot commented Jul 18, 2022

⚠ 无法找到 a-rtifacts 的 CurseForge 项目名到 CurseForge ID 的映射。请检查此链接是否可以正常打开,如果不能,或者网页被重定向,请修改你的文件路径。如果可以,请联系 Cyl18 解决。命令 /add-mapping a-rtifacts [CURSEFORGE_PROJECT_ID]/mv "projects/[GAME_VERSION]/assets/a-rtifacts/" "projects/[GAME_VERSION]/assets/[NEW_DIR]/"
ℹ 此 PR 没有检测到 CurseForge 模组修改。


💾 基于此 PR 所打包的资源包:


⚠ 找不到模组: a-rtifacts-1.16。
ℹ a-rtifacts-1.16 语言文件验证通过。

⚠ 找不到模组: a-rtifacts-1.18。
ℹ a-rtifacts-1.18 语言文件验证通过。


  • 🔄 勾选这个复选框来强制刷新

@cfpa-bot cfpa-bot Bot added the 100+ 行数标签 label Jul 18, 2022
@RCqaq
Copy link
Copy Markdown
Contributor Author

RCqaq commented Jul 18, 2022

我尝试在目录里添加artifacts文件夹但是没有办法添加,加横杠是这个原因吗
好奇为什么a-rtifacts能覆盖artifacts下的语言文件

@Cactusstudent
Copy link
Copy Markdown
Collaborator

为啥CurseForge ID写的是错的呢,应该是artifacts

@Cactusstudent
Copy link
Copy Markdown
Collaborator

我尝试在目录里添加artifacts文件夹但是没有办法添加,加横杠是这个原因吗 好奇为什么a-rtifacts能覆盖artifacts下的语言文件

可以直接更改文件路径

@RCqaq
Copy link
Copy Markdown
Contributor Author

RCqaq commented Jul 21, 2022

可以直接更改文件路径

我就是直接改的,改到artifacts反而不对,所以纳闷
路径名直接灰了,也commit不上去,然后assets/a-rtifacts可以加载,纳闷啊

@Cactusstudent
Copy link
Copy Markdown
Collaborator

可以直接更改文件路径
我就是直接改的,改到artifacts反而不对,所以纳闷

这么离谱?

@RCqaq
Copy link
Copy Markdown
Contributor Author

RCqaq commented Jul 27, 2022

/diff

@cfpa-bot
Copy link
Copy Markdown
Contributor

cfpa-bot Bot commented Jul 27, 2022

@RCqaq 回复内容过长,已经上传至https://gist.github.com/0bc2ea4504a8acbab75a86587bfbefc4

Copy link
Copy Markdown
Contributor

@dovisutu dovisutu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

看起来没有什么问题
artifact难道是与github/git的预留目录名冲突了?

@dovisutu dovisutu added the 即将合并 在此标签被添加的 24 小时后 Pull Request 才可被合并 label Nov 12, 2022
@Cactusstudent Cactusstudent merged commit 12340c8 into CFPAOrg:main Nov 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.16.x Minecraft 版本标签 1.18.x Minecraft 版本标签 100+ 行数标签 Forge 包含 Forge 模组 即将合并 在此标签被添加的 24 小时后 Pull Request 才可被合并

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants