Open
Conversation
Owner
|
感谢铁铁的贡献,给我们补全CI,非常有价值的PR,不过目前项目各平台都还没有非常成熟,判断当前不适合过早引入太多CI,不久一定能用上,再次感谢👍 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
背景
当前仓库在补齐多平台 CI 后,暴露出一批测试对平台细节的隐式假设,尤其是在 Windows 下会受到以下差异影响:
/与\)npm/npm.cmd的可执行名差异RUNNER~1与runneradmin)导致的路径断言失败这次 PR 主要做两件事:补齐跨平台 CI,以及把这批测试改成真正的平台无关。
本次改动
ubuntu-latestwindows-latestmacos-latestVitestubuntu-latest的 model regression 检查npm.cmd相关断言兼容 Windowsopenclaw-package相关测试中 Windows runner 短路径 / 长路径别名造成的误报,改为比较归一化后的真实路径位置验证
本地已验证:
npm test通过npm run typecheck通过npm run build:app通过npm run test:model-regression -- --skip-typecheck通过CI 已验证:
Smoke verify:Ubuntu / Windows / macOS 全通过Vitest:Ubuntu / Windows / macOS 全通过Model regression:Ubuntu 通过影响
这次改动主要是 CI 和测试兼容性增强,不涉及预期的业务行为变更。目标是让后续功能改动能更早在多平台上暴露问题,减少
Windows / macOS 的回归漏检。