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

♻️ 重构工具函数 get_path 与 load_data #19

Merged
merged 1 commit into from
Sep 2, 2023
Merged

♻️ 重构工具函数 get_path 与 load_data #19

merged 1 commit into from
Sep 2, 2023

Conversation

A-kirami
Copy link
Owner

@A-kirami A-kirami commented Sep 2, 2023 •
edited

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

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

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

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

描述

  • get_path 支持获取目录路径
  • get_path 不再检查文件或目录是否存在
  • load_data 移除参数默认值

动机和背景

其他信息

检查工作

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

@A-kirami A-kirami added the enhancement New feature or request label Sep 2, 2023
@A-kirami A-kirami merged commit bc370cf into main Sep 2, 2023
3 checks passed
@A-kirami A-kirami deleted the recycle/path-load branch September 2, 2023 10:37
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
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@A-kirami