Refactor the application package within ruff's complexity limits
Les mainteneurs répondent en général sous 2 jours
Évaluation
- Difficulté
- 4/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 18/100
Piste de recherche
Start by running ruff over policyengine_api/ to list the C901, PLR and PLR2004 violations in the v2 services, Flask routes, data layer, migration logging, ASGI shell and v1 services. Pull request #3933 is already open against this work, so review it before touching any file. Done means ruff reports no violations in those paths, with characterization tests pinning the existing responses, queries, writes and log records.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Phase 1 step 1.8, group 8b, part 2 of 3: refactor the rest of policyengine_api/ (v2 services, Flask routes, the data layer, migration logging and flags, the ASGI shell and the v1 services) so every function is within ruff's complexity limits (C901 and the PLR limits), and name its magic numbers (PLR2004).
Behaviour must not change: the same responses, queries and writes in the same order, log records and published schemas. Flask views, FastAPI routes, Pydantic validators and SQLAlchemy listeners keep their external signatures. Pin behaviour with characterization tests before refactoring.
- Langage dominant
- Python
- Étoiles
- 18
- Forks
- 33
- Merge moyen
- 1 j 19 h
- PR mergées (30 j)
- 26
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-api
-
Difficulté 1/5 1-3 heures Accessibilité débutants 88/100
PolicyEngine/policyengine-api#3911 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 82/100
PolicyEngine/policyengine-api#3910 ·
Les mainteneurs répondent en général sous 2 jours
-
Register weighted-microdata rules: weights aren't valuesPeut-être pris @anth-volk l’a pris il y a 2 jours. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 18/100
PolicyEngine/policyengine-api#3937 ·
Les mainteneurs répondent en général sous 2 jours
-
Remove the unused economy comparison modulePeut-être pris @anth-volk l’a pris il y a 2 jours. Ouverte
Difficulté 3/5 Une demi-journée Accessibilité débutants 18/100
PolicyEngine/policyengine-api#3935 ·
Les mainteneurs répondent en général sous 2 jours
-
Refactor tests, scripts and the rules tool, then select every ruff rule (ALL)Peut-être pris @anth-volk l’a pris il y a 2 jours. Ouverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 15/100
PolicyEngine/policyengine-api#3931 ·
Les mainteneurs répondent en général sous 2 jours
Toutes les issues de PolicyEngine/policyengine-api
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
PolicyEngine/policyengine-us#10262 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
kellyjonbrazil/jc#788 ·
Les mainteneurs répondent en général sous 2 jours
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
`/search.json` ignores the documented `editions.sort` parameterPeut-être pris @PranavD2905 l’a pris aujourd’hui. OuverteAffects: Server Good First Issue Module: Solr Needs: Triage Theme: Public APIs Theme: Search
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
internetarchive/openlibrary#13989 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
bug P2: misleading query
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
jeffknupp/association#478 ·