Use loguru and appdirs to replace logging and hard coded path #82
Conversation
zrr1999
commented
Oct 14, 2023
•
zrr1999
commented
- 使用 loguru 代替 logging，并将compiler中的print替换为日志形式。
- 使用 appdirs 获取缓存路径，避免为每个系统单独寻找缓存路径。
- 移除 isort 并在 ruff 中加入相应替代配置。
| rev: 5.12.0 | ||
| hooks: | ||
| - id: isort | ||
| - repo: https://github.com/psf/black |
There was a problem hiding this comment.
ruff-format 目前可以替代 black 吗?
Using Ruff's formatter (unstable)
https://github.com/astral-sh/ruff-pre-commit#using-ruffs-formatter-unstable
这里是 "unstable" 的状态, 另外这里需要加上
...
hooks:
- id: ruff-format
吗?
There was a problem hiding this comment.
这块我也不太确定，我把autoflake的配置给恢复回去了
There was a problem hiding this comment.
建议 暂时保留 black 的配置, 可以在后面替换(如果有必要)
There was a problem hiding this comment.
另外这里需要加上 ... 吗？
需要的，ruff 和 ruff-format 是两个不同的 hook，ruff 仅仅包含 lint 而不包含 format（https://github.com/astral-sh/ruff-pre-commit/blob/42f98979dbdfcd148dff424477552b8816a7cf01/.pre-commit-hooks.yaml#L4 ）
format 还用 black 吧，ruff 还太早期了 0.0.291 还只是第一个用户可见的版本，距离可用还需要很久的打磨
不过我相信 isort 已经可以替换了，已经打算 ruff 过两天发布 0.1.0 （astral-sh/ruff#7931）就在 Paddle 那边将 isort 替换为 ruff 的 isort 实现
| rev: 5.12.0 | ||
| hooks: | ||
| - id: isort | ||
| - repo: https://github.com/psf/black |
There was a problem hiding this comment.
另外这里需要加上 ... 吗？
需要的，ruff 和 ruff-format 是两个不同的 hook，ruff 仅仅包含 lint 而不包含 format（https://github.com/astral-sh/ruff-pre-commit/blob/42f98979dbdfcd148dff424477552b8816a7cf01/.pre-commit-hooks.yaml#L4 ）
format 还用 black 吧，ruff 还太早期了 0.0.291 还只是第一个用户可见的版本，距离可用还需要很久的打磨
不过我相信 isort 已经可以替换了，已经打算 ruff 过两天发布 0.1.0 （astral-sh/ruff#7931）就在 Paddle 那边将 isort 替换为 ruff 的 isort 实现
pyproject.toml
Outdated
| ] | ||
| ignore = [ | ||
| "UP015", | ||
| "Q000", |
There was a problem hiding this comment.
这里没有 select Q 为什么要 ignore Q000 呢？
| - repo: https://github.com/psf/black-pre-commit-mirror | ||
| rev: 23.9.1 |
There was a problem hiding this comment.
将 black 加回来并更新为最新版本和使用 mirror 版本（就相当于从 PyPI 下载而不是 clone 源码构建），因为 mirror 版本比 source 版本的优势是通过 mypyc 编译，有更快的速度，也是现在推荐的方式，相关讨论见 psf/black#3405 ，文档见 https://black.readthedocs.io/en/stable/integrations/source_version_control.html
