testcontainers / testcontainers/testcontainers-python
Feature: Use pre-commit.ci
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 2.3k
- Forks
- 386
- Merge moyen
- 4 h 40 min
- PR mergées (30 j)
- 1
Description
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.
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par examiner le workflow CI actuel et la configuration pre-commit du dépôt, en vous concentrant sur les hooks listés et les commandes mypy distinctes. Confirmez comment l’option de saut de CI de pre-commit.ci s’intègre au workflow existant ; le travail est terminé lorsque les hooks sont définis de manière cohérente pour une utilisation locale et dans CI, et que mypy s’exécute toujours correctement.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- github-actions, python
- Domaine
- ci-cd
- Type d'issue
- Fonctionnalité
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- Calme
- Clarté
- Plutôt claire
- Accessibilité débutants
- 55/100