As a developer, I have an easy way to check my commit and code locally before pushing
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 35/100
- Type d'issue
- Fonctionnalité
- Clarté
- À clarifier
- Activité
- À l'abandon
- Stack technique
- python
- Domaine
- ci-cd, developer-experience, tooling
Piste de recherche
Start by reviewing the existing CI checks for commit validation, style and lint checks, and changelog handling, then compare the linked pre-commit examples. Done means developers have an easy local command or workflow that runs the relevant checks before pushing.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Author: daviddavis (daviddavis)
Redmine Issue: 6938, https://pulp.plan.io/issues/6938
I was talking to ggainey about some of the automation checks we have in pulp (commit validation, style/lint checks, changelog, etc) and how it might be frustrating for new devs. I've heard the same feedback from other contributors such as galaxy.
We need to provide an easy way for developers to run our CI checks locally. I've seen other projects (bandersnatch[0], black[1], celery[2], etc) use pre-commit:
[0] https://github.com/pypa/bandersnatch/blob/master/.pre-commit-config.yaml
[1] https://github.com/psf/black/blob/master/.pre-commit-config.yaml
[2] https://github.com/celery/celery/blob/master/.pre-commit-config.yaml
- Langage dominant
- Python
- Étoiles
- 10
- Forks
- 40
- Merge moyen
- 8 h 59 min
- PR mergées (30 j)
- 5
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire 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.
Autres issues de pulp/plugin_template
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
pulp/plugin_template#724 · 1 commentaire ·
-
stale
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
pulp/plugin_template#444 · 1 commentaire · 1 réaction ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
pulp/plugin_template#1040 · 1 commentaire ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
pulp/plugin_template#1027 · 4 commentaires ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 55/100
pulp/plugin_template#973 ·
Toutes les issues de pulp/plugin_template
Issues similaires
-
[Bug] @deck.gl/arcgis dist import resolves to unpublished @deck.gl/core source path (9.3.11, 9.4.0)Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
workflow: a tick's dispatch counts as 'only this step', and no review self-grants a round unattendedOuverteworkflow
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
kristofdegrave/homeassistant-smart-charging#1505 ·
Les mainteneurs répondent en général sous 1 jour
-
New Submission: TropWATEROuvertemetadata submission
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
-
Wrongly named dashboard variableOuvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
canonical/content-cache-operator#163 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
[submission]Ouvertesubmission
Difficulté 1/5 Moins d'une heure Accessibilité débutants 65/100
leanprover/lean-eval-submissions#1852 ·
Les mainteneurs répondent en général sous 1 jour