akka/akka-http

Use override keyword more prominently in Http codebase

Offen

#72 geöffnet am 08.09.2016

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Scala (598 Forks)batch import
1 - triagedhelp wantednice-to-have (low-prio)

Repository-Metriken

Stars
 (1.311 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 1T 10h) (2 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Identifiziere alle Methodendefinitionen, die eine Methode einer Superklasse oder eines Traits überschreiben, aber den 'override' Modifikator vermissen, und füge das Schlüsselwort hinzu.
Tech Stack
scala
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitScala
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70