hust-open-atom-club / linux-edu-rank

Linux 内核高校贡献排名榜
https://edurank.hust.openatom.club
BSD 2-Clause "Simplified" License
2 stars 6 forks source link

feat: add pylint CI check #23

Closed showlibia closed 1 month ago

tttturtle-russ commented 1 month ago

Thanks for the PR. Can we add a minimum score, like 8.5 or 9? Only when pylint scores higher than the minimum score can pass the CI.

jingfelix commented 1 month ago

我建议添加一份 pre-commit 配置,把 pylint 作为 pre-commit hook,在 Action 中通过 pre-commit 调用,优点是:

  1. 本地提交前直接检查
  2. 可以用上 pre-commit workflow 中配置好的缓存机制

ref: Pre-commit integration a GitHub action to run pre-commit

[!NOTE] 这里应该不需要 skip: [pylint]