Skip to content

Conversation

@Mikachu2333
Copy link
Collaborator

问题描述

  1. 增加大量函数描述 #259 (comment)
  2. N/A

方案与实现

  1. 这两个函数都直接转换为linux的路径分隔符(/)并进行计算, 仅在最后统一转换为适合win/linux的路径分隔符
  2. 增加了对单独一个 ~ 的处理
  3. 修复了父目录对于以 \ 结尾的目录处理有误的问题
  4. 增加了一堆tests

@github-actions
Copy link

Hi @Mikachu2333,

❤️ 感谢你的贡献!你的 PR 当前基于 main 分支,请修改使用 dev 分支

@Mikachu2333 Mikachu2333 changed the base branch from main to dev August 17, 2025 11:56
@Mikachu2333
Copy link
Collaborator Author

在考虑是否应当加入一个ignore case在路径标准化里面,因为路径是大小写不敏感的,并且确实可能存在这种可能——误用了错误的大小写

@ccmywish
Copy link
Contributor

这个实现的质量是可以的 👍👍👍

@ccmywish
Copy link
Contributor

在考虑是否应当加入一个ignore case在路径标准化里面,因为路径是大小写不敏感的,并且确实可能存在这种可能——误用了错误的大小写

不应该,“路径的大小写不敏感” 只在Windows上成立,但是显然 chsrc 的维护者要负责维护多个平台,不应该在其他平台保持大小写警惕的时候只在 Windows 上随意。所以如果写错了就是维护者的错误。

@ccmywish ccmywish merged commit 4d81389 into RubyMetric:dev Aug 18, 2025
@Mikachu2333 Mikachu2333 deleted the fix/incorrect_path branch August 18, 2025 11:03
@Mikachu2333
Copy link
Collaborator Author

不应该,“路径的大小写不敏感” 只在Windows上成立,但是显然 chsrc 的维护者要负责维护多个平台,不应该在其他平台保持大小写警惕的时候只在 Windows 上随意。所以如果写错了就是维护者的错误。

有道理

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