Ruff lints the whole repo
Les mainteneurs répondent en général sous 5 jours
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 68/100
Piste de recherche
Commencez par .github/workflows/linters.yaml et examinez l’étape existante Check code style, puis ouvrez une pull request pour vérifier quels fichiers Ruff vérifie actuellement. Mettez à jour le workflow afin que les commandes de formatage et de lint de Ruff ne reçoivent que les fichiers Python modifiés, et confirmez que la vérification est ignorée proprement lorsqu’aucun fichier Python n’a été modifié.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
ℹ️ Please note that the best way to get support for Real Python courses & articles is to join one of our weekly Office Hours calls or in the RP Community Slack.
You can report issues and problems here, but we typically won't be able to provide 1:1 support outside the channels listed above.
Describe the bug
Currently, Ruff lints all the files in the repo, which can be inefficient.
To Reproduce
Steps to reproduce the behavior:
- Open a PR and take a look at the Ruff check details.
Expected behavior
Lint only the modified Python files.
Additional context
Proposed update to file .github/workflows/linters.yaml
- name: Check code style
run: |
. venv/bin/activate
if [ "$GITHUB_EVENT_NAME" = "pull_request" ]; then
git fetch origin ${{ github.base_ref }}
CHANGED_FILES=$(git diff --name-only origin/${{ github.base_ref }}...${{ github.sha }} -- '*.py')
else
CHANGED_FILES=$(git diff --name-only HEAD^ -- '*.py')
fi
if [ -n "$CHANGED_FILES" ]; then
python -m ruff format --check $CHANGED_FILES
python -m ruff check $CHANGED_FILES
fi
- Langage dominant
- Jupyter Notebook
- Étoiles
- 5.2k
- Forks
- 5.3k
- Merge moyen
- 5 j 4 h
- PR mergées (30 j)
- 15
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Aucun 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 realpython/materials
-
пOuverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 1/100
realpython/materials#763 ·
Les mainteneurs répondent en général sous 5 jours
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 55/100
realpython/materials#738 ·
Les mainteneurs répondent en général sous 5 jours
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 45/100
realpython/materials#737 ·
Les mainteneurs répondent en général sous 5 jours
-
проект2Ouverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 15/100
realpython/materials#725 ·
Les mainteneurs répondent en général sous 5 jours
-
pip install -r requirements.txtOuverte
Difficulté 3/5 1-2 jours Accessibilité débutants 15/100
realpython/materials#721 ·
Les mainteneurs répondent en général sous 5 jours
Toutes les issues de realpython/materials
Issues similaires
-
Difficulté 1/5 1-3 heures Accessibilité débutants 88/100
-
backend tests
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
lognorman20/monaco#2223 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 90/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
community-scripts/ProxmoxVE#17674 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
EPFL-ENAC/co2-calculator#3007 ·
Les mainteneurs répondent en général sous 1 jour