akka/akka-http

Use override keyword more prominently in Http codebase

Aperta

#72 aperta il 8 set 2016

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)Scala (598 fork)batch import
1 - triagedhelp wantednice-to-have (low-prio)

Metriche repository

Star
 (1311 stelle)
Metriche merge PR
 (Merge medio 1g 10h) (2 PR mergiate in 30 g)

Descrizione

Issue by ktoso Wednesday Mar 09, 2016 at 15:08 GMT Originally opened as https://github.com/akka/akka/issues/19986

In general the HTTP codebase does not use the override keyword too much. AFAIR we had some discussions and were moving towards using it more, anyway most important is to be consistent so this should be applied over the entire codebase.

One instance of such places here: https://github.com/akka/akka/pull/19984

Guida contributor

Direzione di ricerca
Identifica tutte le definizioni di metodo che sovrascrivono un metodo di superclasse o trait ma mancano del modificatore 'override', quindi aggiungi la parola chiave.
Tech stack
scala
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitScala
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70