imbajin commented on code in PR #313:
URL:
https://github.com/apache/incubator-hugegraph-ai/pull/313#discussion_r2566997919
##########
.github/workflows/ruff.yml:
##########
@@ -1,20 +1,18 @@
-# TODO: replace by ruff & mypy soon
-name: "Pylint"
+name: "Ruff Code Quality"
on:
push:
branches:
- - 'main'
- - 'master'
- - 'release-*'
+ - "main"
+ - "release-*"
pull_request:
Review Comment:
‼️ **关键问题：移除了 master 分支的 CI 触发**
原配置中包含 `master` 分支的 CI 触发，但新配置中移除了。如果项目仍在使用 `master` 分支，这可能导致 master
分支上的提交不再运行 lint 检查。
请确认这是有意为之（例如项目已完全迁移到 `main`），还是遗漏？
```suggestion
push:
branches:
- "main"
- "master"
- "release-*"
```
##########
pyproject.toml:
##########
@@ -43,6 +43,9 @@ dev = [
"pytest~=8.0.0",
"pytest-cov~=5.0.0",
"pylint~=3.0.0",
Review Comment:
⚠️ **建议：保留 pylint 或明确说明迁移计划**
虽然 ruff 可以替代大部分 pylint 功能，但 pylint 仍然有一些独特的检查规则（如更深入的类型推断和复杂的代码分析）。当前配置同时保留了
pylint 和 ruff：
1. 如果打算用 ruff 完全替代 pylint，建议在后续 PR 中移除 pylint 依赖
2. 如果需要两者并存，建议在文档中说明各自的职责
另外，注意当前代码中还有大量 `# pylint: disable=...` 注释，迁移到 ruff 后这些注释将失效（不会报错但也不起作用）。
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
