Is it good to use `reverse_merge!` on the optional hash argument of a method?
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 25/100
- Tipo de issue
- Documentación
- Claridad
- Necesita aclaración
- Estado de actividad
- Estancado
- Stack tecnológico
- ruby
- Área
- documentation
Línea de trabajo
Comienza localizando la guía que contiene el ejemplo de obliterate y revisa cómo se tratan los argumentos hash opcionales y reverse_merge!. Comprueba la documentación circundante y cualquier guía de estilo relacionada; se considera terminado cuando la pregunta tiene una recomendación definida y la guía la refleja claramente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
I found an example in this guide
def obliterate(things, options = {})
default_options = {
:gently => true, # obliterate with soft-delete
:except => [], # skip obliterating these things
:at => Time.now, # don't obliterate them until later
}
options.reverse_merge!(default_options)
...
end
I'm wondering whether this is a good practice. When you use this method, the input options hash might be modified by reverse_merge! method, which, in my opinion, is very error-prone process in a big project.
- Lenguaje dominante
- Ruby
- Estrellas
- 3.9k
- Forks
- 757
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- 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 airbnb/ruby
-
Warnings about rule namespaces for Capybara and FactoryBotPosiblemente ocupada @mitorez la tomó hace 138 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 58/100
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 10/100
-
[bug] Update rubocop to 1.6.2Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 48/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
Todos los issues de airbnb/ruby
Issues similares
-
bug f:search
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
feature
Dificultad 2/5 1-3 horas Aptitud para principiantes 60/100
manyfold3d/manyfold#7346 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
🐞 bug pending triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
CircuitVerse/CircuitVerse#8025 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
we-promise/sure#4199 ·
Los mantenedores suelen responder en 1 día
-
Zibi version number updateAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
riscv/riscv-unified-db#2738 ·
Los mantenedores suelen responder en 1 día