Skip to content

refactor: 重构构建系统 - #110

Closed
kzyqq00-Player wants to merge 11 commits into
xBoyMinemc:mainfrom
kzyqq00-Player:refactor-builder
Closed

refactor: 重构构建系统#110
kzyqq00-Player wants to merge 11 commits into
xBoyMinemc:mainfrom
kzyqq00-Player:refactor-builder

Conversation

@kzyqq00-Player

@kzyqq00-Player kzyqq00-Player commented Aug 15, 2025

Copy link
Copy Markdown
Contributor

待办

  • 每次更新不用猥琐的改代码
  • 是否releasejs控制而不是workflow
  • 整理石山代码

@xBoyMinemc

Copy link
Copy Markdown
Owner

新增的缓存检查、manifest语法校验是不被需要的
构建不应当考虑缓存
原先的manifest的构造方式决定了它的正确
release与否,应当workflow控制,安全且可控,控制信息显露于提交信息中,无遮盖或间接影响。
虽然当前通过tags控制未明确列在不存在的技术文档上,但依旧应当保留这个行为。

@kzyqq00-Player

Copy link
Copy Markdown
Contributor Author

新增的缓存检查、manifest语法校验是不被需要的 构建不应当考虑缓存 原先的manifest的构造方式决定了它的正确 release与否,应当workflow控制,安全且可控,控制信息显露于提交信息中,无遮盖或间接影响。 虽然当前通过tags控制未明确列在不存在的技术文档上,但依旧应当保留这个行为。

我也考虑过这个问题,但是“缓存”不应该叫缓存,只要你仔细读一遍就会发现,我的目的是保存上一版本的信息。
还有另一个方案,就是从manifest.json中读取,但是manifest.json里面的字符串格式多变,而js用严格的格式来读取的话如果不符合格式,那我数据从哪来?

@kzyqq00-Player

Copy link
Copy Markdown
Contributor Author

或者加一个开发文档,但这样治标不治本。

@kzyqq00-Player

Copy link
Copy Markdown
Contributor Author

又又又又有一个方案,就是js和workflow都可以控制是否release,什么版本。

@xBoyMinemc
xBoyMinemc requested a review from PuppyOne August 16, 2025 09:17
@kzyqq00-Player

This comment was marked as resolved.

@kzyqq00-Player

Copy link
Copy Markdown
Contributor Author

完了卧槽,我新写的代码跟托💩一样,看来这个pr要废了

@kzyqq00-Player

Copy link
Copy Markdown
Contributor Author

看来也没啥用,我close了

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