Refactor within ruff's complexity limits, then select every ruff rule (ALL)
Mantenedores costumam responder em até 1 dia
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 15/100
Direção de pesquisa
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.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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).
- Linguagem predominante
- Python
- Estrelas
- 1
- Forks
- 1
- Merge médio
- 1d 11h
- PRs com merge (30d)
- 50
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-sim-api
-
Integration tests fail due to missing UV virtual environment setupTalvez já em andamento Um pull request vinculado a esta issue está aberto ou já foi mesclado. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
PolicyEngine/policyengine-sim-api#314 ·
Mantenedores costumam responder em até 1 dia
-
enhancement
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 22/100
PolicyEngine/policyengine-sim-api#814 ·
Mantenedores costumam responder em até 1 dia
-
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 30/100
PolicyEngine/policyengine-sim-api#806 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 25/100
PolicyEngine/policyengine-sim-api#805 ·
Mantenedores costumam responder em até 1 dia
-
Enable ruff argument, idiom and pytest-style rules (ARG, B, C4, FBT, PERF, PLW, PT, PTH, RUF, SIM and others)Talvez já em andamento @anth-volk assumiu há 1 dia. Aberta
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 22/100
PolicyEngine/policyengine-sim-api#801 ·
Mantenedores costumam responder em até 1 dia
Todas as issues de PolicyEngine/policyengine-sim-api
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
huggingface/sentence-transformers#4186 ·
Mantenedores costumam responder em até 5 dias
-
documentation good first issue
Dificuldade 2/5 Meio dia Facilidade para iniciantes 72/100
sujithputta02/Esapay#13 ·
-
bot-triaged module: correctness (silent) module: mps module: NaNs and Infs triage review
Dificuldade 2/5 Meio dia Facilidade para iniciantes 62/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
Mantenedores costumam responder em até 1 dia
-
README: CI link/badge checkAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 62/100
SCIInstitute/SCIRun#2842 ·
Mantenedores costumam responder em até 3 dias