Skip to content

Conversation

Mikachu2333
Copy link
Collaborator

Copy link

Hi @Mikachu2333

❤️ 感谢你的贡献!我们将在最少半小时,最多5天内阅读此 PR 并回复你

@Mikachu2333
Copy link
Collaborator Author

@ccmywish on_android 似乎从未被调用过

@Mikachu2333
Copy link
Collaborator Author

此外,我认为有必要为兼容性考虑,将bsd的on_linux值设为true

mac没用过不知啥情况

@ccmywish
Copy link
Contributor

这个函数就是为了直接区分这几个系统,恰恰和你说的 “为兼容性考虑” 是相反的。

detect_os() 从命名本身看就是为了检测出不同的系统,最终检测都是 on_linux 那这个函数都没有啥必要了。

你说的那种兼容性,可以引入 额外的 on_unix_family 来处理。

Copy link
Contributor

@ccmywish ccmywish left a comment

Choose a reason for hiding this comment

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

  1. 恢复检测 Linux 本身的方式,保留你对安卓的检测方式的修改,显著区分二者。

  2. 不要用 REF,用代码里每一个 recipe 使用的 @consult 统一标准

@ccmywish ccmywish added this to the v0.2.4 milestone Oct 15, 2025
@Mikachu2333
Copy link
Collaborator Author

之前linux检测逻辑消失是手滑一不小心多删了导致的(

@ccmywish ccmywish merged commit 6af3e51 into RubyMetric:dev Oct 15, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants