Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

🐛 修复 File name too long 错误 #32

Merged
merged 1 commit into from
Sep 10, 2023
Merged

🐛 修复 File name too long 错误 #32

merged 1 commit into from
Sep 10, 2023

Conversation

A-kirami
Copy link
Owner

这个 PR 带来了什么样的更改？

  • 错误修复
  • 新功能
  • 文档/注释
  • 代码格式
  • 代码重构
  • 测试用例
  • 性能优化
  • 外观样式
  • 项目构建
  • 依赖环境
  • 持续集成/部署
  • 其他，请描述:

这个 PR 是否存在破坏性变更？

  • 是的，并已在 issue #___ 号中获得批准
  • 没有

描述

不再使用 Path 判断字符串是否为路径，通过传入的 str 和 Path 类型来判断是字符串还是路径

动机和背景

在 Linux 上使用 Path 判断字符串可能会引发 File name too long 错误

其他信息

检查工作

  • 我对我的代码进行了注释，特别是在难以理解的部分
  • 我的更改需要更新文档，并且已对文档进行了相应的更改
  • 我添加了测试并且已经在本地通过，以证明我的修复补丁或新功能有效
  • 我已检查并确保更改没有与其他打开的 Pull Requests 重复

@A-kirami A-kirami added the bug Something isn't working label Sep 10, 2023
@A-kirami A-kirami merged commit e2cb682 into main Sep 10, 2023
3 checks passed
@A-kirami A-kirami deleted the fix/filename-too-long branch September 10, 2023 06:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@A-kirami