Refactor within ruff's complexity limits, then select every ruff rule (ALL)
Les mainteneurs répondent en général sous 1 jour
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 15/100
Piste de recherche
The issue names no files. Start by running ruff with the complexity rules (C901 and the PLR limits) enabled to list the functions over the limits, and check the ruff config in the repository for the current selection. Before touching any function, add characterization tests that pin its outputs, errors, logs, and OpenAPI output. Done means the listed functions are under the limits, behaviour is unchanged, and ruff passes with select = ["ALL"] and only the stated global ignores.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Phase 1 step 1.8, group 8b, the last group: refactor every function over ruff's complexity limits (C901, and the PLR limits on arguments, branches, statements and returns), name the magic numbers (PLR2004), then switch ruff to select = ["ALL"].
Behaviour must not change. That covers the same outputs, errors, logs, Modal function signatures, library APIs other projects import, and byte-identical OpenAPI documents. Pin behaviour with characterization tests before each refactor.
The only global ignores are the two rules that conflict with ruff format (COM812, ISC001) and one rule of each mutually exclusive pydocstyle pair (D203, D213).
- Langage dominant
- Python
- Étoiles
- 1
- Forks
- 1
- Merge moyen
- 1 j 11 h
- PR mergées (30 j)
- 50
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 PolicyEngine/policyengine-sim-api
-
Integration tests fail due to missing UV virtual environment setupPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
PolicyEngine/policyengine-sim-api#314 ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement
Difficulté 4/5 3-5 jours Accessibilité débutants 22/100
PolicyEngine/policyengine-sim-api#814 ·
Les mainteneurs répondent en général sous 1 jour
-
Register weighted-microdata rules: weights aren't valuesPeut-être pris @anth-volk l’a pris il y a 1 jour. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 30/100
PolicyEngine/policyengine-sim-api#806 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 25/100
PolicyEngine/policyengine-sim-api#805 ·
Les mainteneurs répondent en général sous 1 jour
-
Enable ruff argument, idiom and pytest-style rules (ARG, B, C4, FBT, PERF, PLW, PT, PTH, RUF, SIM and others)Peut-être pris @anth-volk l’a pris il y a 2 jours. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 22/100
PolicyEngine/policyengine-sim-api#801 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de PolicyEngine/policyengine-sim-api
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
UKGovernmentBEIS/control-arena#917 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
debpalash/VoiceStudio#2937 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
aws-samples/sample-per-model-bedrock#261 ·
Les mainteneurs répondent en général sous 1 jour