Skip to content

Conversation

@Mikachu2333
Copy link
Collaborator

@Mikachu2333 Mikachu2333 commented Aug 7, 2025

问题描述

  1. 路径获取错误
  2. fix Windows上获取 “系统 Documents” 目录的方式有问题 #245

方案与实现

  1. 增加了一堆路径处理专用函数,部分替换了以前的函数(屎山喜加一,并且出于兼容性考虑以及我不精通c,之前的很多路径处理的地方都没敢完全替换)
  2. 彻底修正了test
  3. 我真是草了,怎么还是会自动格式化啊……
  4. 不确定能不能用,我目前测着没啥问题(

@github-actions
Copy link

github-actions bot commented Aug 7, 2025

Hi @Mikachu2333,

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

@Mikachu2333 Mikachu2333 changed the base branch from main to dev August 7, 2025 11:14
Mikachu2333 added 2 commits August 7, 2025 19:25
@Mikachu2333
Copy link
Collaborator Author

看起来还有问题……

@Mikachu2333
Copy link
Collaborator Author

Mikachu2333 commented Aug 7, 2025

问一下 @ccmywish ,我对path的改动会影响 backup函数吗?现在所有的path只要被规范化过就会自动在开头和结尾加单引号

其余的我都检查了一下,没发现问题,就这个backup拿不准

@ccmywish ccmywish closed this Aug 7, 2025
@ccmywish
Copy link
Contributor

ccmywish commented Aug 7, 2025

image image

这太夸张了,每次提交一下都会发送一封邮件。你应该在自己的仓库确保完全可以工作之后再提交 PR

@ccmywish
Copy link
Contributor

ccmywish commented Aug 7, 2025

另外,这个改动太大了。肯定要分开。

@Mikachu2333
Copy link
Collaborator Author

另外,这个改动太大了。肯定要分开。

这咋分,确实是一整个大功能改进(

@Mikachu2333
Copy link
Collaborator Author

这太夸张了,每次提交一下都会发送一封邮件。你应该在自己的仓库确保完全可以工作之后再提交 PR

ok

@ccmywish
Copy link
Contributor

ccmywish commented Aug 7, 2025

你的代码格式一直没有规范下来,所以如果你提交一大块进来,我得挨个修改。请务必务必务必要跟周围的代码格式一样。

  1. 函数调用都要和括号中间有一个空格 func (
  2. 任何语句结构的 { 都要单独起一行,并再缩进2格

@ccmywish
Copy link
Contributor

ccmywish commented Aug 7, 2025

这咋分,确实是一整个大功能改进(

尽量分,实在分不了再说

@Mikachu2333 另外,我看你也工作了一天了,辛苦了 👍

维护开源项目是一个细致缓慢地过程,不要心急,一次只做一点工作,这样身体才能可持续地发展,项目也能持续稳步进展。先早点休息吧 🤝

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