Lint/NoReturnInMemoization flags `return` in local variable `||=` and `&&=` assignments
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 84/100
Línea de trabajo
El issue identifica on_or_asgn y su alias on_and_asgn como los puntos de entrada relevantes y menciona las pruebas de asignación relacionadas. Lee la implementación del cop y esas pruebas, y añade cobertura para variables locales a las que se asigna ||= y &&=. Ejecuta las pruebas enfocadas del cop; el trabajo estará terminado cuando estos casos no generen infracciones y el comportamiento existente para variables de instancia siga pasando.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Lint/NoReturnInMemoization reports an offense for a return inside a begin..end block assigned to a local variable with ||= or &&=:
# frozen_string_literal: true
def foo
x ||= begin
return 1 if bar
2
end
x
end
$ bundle exec rubocop --only Lint/NoReturnInMemoization sample.rb
sample.rb:5:5: W: Lint/NoReturnInMemoization: Do not return in begin..end blocks in instance variable assignment contexts.
Expected behavior
No offense. The cop description and message cover instance variable assignment, and the cop already ignores the same code with x = begin ... end or x += begin ... end (see the test_returns_in_regular_assignment_are_not_offenses and test_returns_in_operation_assignment_for_instance_variable_are_not_offenses tests). A local variable goes out of scope when the method returns, so no memoized value is lost.
Cause
on_op_asgn returns early unless the target is an instance variable, but on_or_asgn (also aliased as on_and_asgn) checks every target.
Version: rubocop-shopify main (3.1.0), rubocop 1.92.0.
- Lenguaje dominante
- Ruby
- Estrellas
- 638
- Forks
- 50
- Merge medio
- 7 h 26 min
- PR fusionados (30 d)
- 3
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de Shopify/ruby-style-guide
-
Include reasoning in some rulesAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
Shopify/ruby-style-guide#190 · 1 reacción ·
Todos los issues de Shopify/ruby-style-guide
Issues similares
-
[BUG]: Export Donations button state not updatingPosiblemente ocupada @yashiyadav la tomó hoy. Abierto🐞 Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
rubyforgood/human-essentials#5744 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
oven-sh/bun#46525 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
zerocracy/judges-action#2849 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
bigbluebutton/bigbluebutton#26166 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
TarteelAI/quranic-universal-library#916 ·
Los mantenedores suelen responder en 1 día