akka/akka-http

Use override keyword more prominently in Http codebase

Ouverte

#72 ouverte le 8 sept. 2016

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)Scala (598 forks)batch import
1 - triagedhelp wantednice-to-have (low-prio)

Métriques du dépôt

Stars
 (1 311 étoiles)
Métriques de merge PR
 (Merge moyen 1j 10h) (2 PRs mergées en 30 j)

Description

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

Guide contributeur

Direction de recherche
Identifiez toutes les définitions de méthodes qui remplacent une méthode de superclasse ou de trait et qui manquent du modificateur 'override', puis ajoutez le mot clé.
Stack technique
scala
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitScala
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70