testcontainers / testcontainers/testcontainers-python
Feature: Use pre-commit.ci
未关闭
还没有人认领这个 Issue。
🚀 enhancement
- 主要语言
- Python
- 星标
- 2.3k
- 派生
- 386
- 平均合并
- 4 小时 40 分钟
- 30 天内合并 PR
- 1
描述
Currently we use the following in our ci workflow
- name: Execute pre-commit handler
continue-on-error: true
run: |
poetry run pre-commit run check-toml
poetry run pre-commit run trailing-whitespace
poetry run pre-commit run end-of-file-fixer
poetry run pre-commit run ruff
poetry run pre-commit run ruff-format
- name: Execute mypy
run: |
make mypy-core-report
make mypy-core
So we run all hooks but the mypy hool and then run mypy.
I suggest we use https://pre-commit.ci/
This service run the pre-commit hooks and offers a skip option for ci, so we have one file where to define which hooks should run in ci and locally.
It also has speed advantages and can commit changes of ruff back to MR.
It also updates the pre-commit file automatically, creating a PR.
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
首先检查当前的 CI 工作流和仓库的 pre-commit 配置,重点关注列出的 hook 以及单独的 mypy 命令。确认 pre-commit.ci 的 CI 跳过选项如何适配现有工作流;当这些 hook 已针对本地使用和 CI 使用进行一致定义,且 mypy 仍能成功运行时,即视为完成。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- github-actions, python
- 领域
- ci-cd
- Issue 类型
- 功能
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 活跃度
- 冷清
- 描述清晰度
- 基本清楚
- 新手友好度
- 55/100