Refactor the application package within ruff's complexity limits
Mantenedores costumam responder em até 2 dias
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- Mais de uma semana
- Facilidade para iniciantes
- 18/100
Direção de pesquisa
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.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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.
- Linguagem predominante
- Python
- Estrelas
- 18
- Forks
- 33
- Merge médio
- 1d 19h
- PRs com merge (30d)
- 26
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de PolicyEngine/policyengine-api
-
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 88/100
PolicyEngine/policyengine-api#3911 ·
Mantenedores costumam responder em até 2 dias
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 82/100
PolicyEngine/policyengine-api#3910 ·
Mantenedores costumam responder em até 2 dias
-
Register weighted-microdata rules: weights aren't valuesTalvez já em andamento @anth-volk assumiu há 1 dia. Aberta
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 18/100
PolicyEngine/policyengine-api#3937 ·
Mantenedores costumam responder em até 2 dias
-
Remove the unused economy comparison moduleTalvez já em andamento @anth-volk assumiu há 1 dia. Aberta
Dificuldade 3/5 Meio dia Facilidade para iniciantes 18/100
PolicyEngine/policyengine-api#3935 ·
Mantenedores costumam responder em até 2 dias
-
Refactor tests, scripts and the rules tool, then select every ruff rule (ALL)Talvez já em andamento @anth-volk assumiu há 1 dia. Aberta
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 15/100
PolicyEngine/policyengine-api#3931 ·
Mantenedores costumam responder em até 2 dias
Todas as issues de PolicyEngine/policyengine-api
Issues semelhantes
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
jbacalso24/mimry#291 ·
Mantenedores costumam responder em até 1 dia
-
good first issue
Dificuldade 2/5 Menos de uma hora Facilidade para iniciantes 82/100
eli-alvarado/us-watchlist-alerts#14 ·
Mantenedores costumam responder em até 1 dia
-
from-review-extraction python severity:nit
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 80/100
LearningCircuit/local-deep-research#7462 ·
Mantenedores costumam responder em até 2 dias
-
mermaid_extract.py silently discards single-line state-diagram notes; sequence notes are keptAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
cathrynlavery/diagram-design#441 ·
Mantenedores costumam responder em até 2 dias
-
[Cleanup] Include the public header in work_split.cppTalvez já em andamento @blozano-tt assumiu hoje. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
tenstorrent/tt-metal#60850 ·
Mantenedores costumam responder em até 1 dia