Is it good to use `reverse_merge!` on the optional hash argument of a method?
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 25/100
- Tipo di issue
- Documentazione
- Chiarezza
- Da chiarire
- Stato di attività
- Ferma
- Stack tecnologico
- ruby
- Ambito
- documentation
Direzione di ricerca
Inizia individuando la guida che contiene l'esempio con obliterate e verifica come vengono trattati gli argomenti hash opzionali e reverse_merge!. Controlla la documentazione circostante e le eventuali indicazioni di stile correlate; il lavoro è concluso quando la domanda ha una raccomandazione definita e la guida la riflette chiaramente.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- Ruby
- Stelle
- 3.9k
- Fork
- 757
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di airbnb/ruby
-
Warnings about rule namespaces for Capybara and FactoryBotForse già presa @mitorez l’ha presa 138 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 58/100
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 10/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 48/100
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 45/100
Issue simili
-
bug f:search
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
feature
Difficoltà 2/5 1-3 ore Idoneità per principianti 60/100
manyfold3d/manyfold#7346 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
🐞 bug pending triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
CircuitVerse/CircuitVerse#8025 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
we-promise/sure#4199 ·
I maintainer di solito rispondono entro 1 giorno
-
Zibi version number updateAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
riscv/riscv-unified-db#2738 ·
I maintainer di solito rispondono entro 1 giorno