Inline Method refactoring is allowed in abstract methods.
Les mainteneurs répondent en général sous 7 jours
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 45/100
Piste de recherche
Start by reproducing the issue with the Base.tag() snippet in the report, then find the Inline Method refactoring entry point in rope's refactoring code. The report doesn't name the file, so search for it. Done means applying Inline Method to an abstract method warns or refuses, and a test covers the snippet. The report says 'warning' but doesn't say whether it should be an error, so confirm with a maintainer before choosing.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Abstract methods do not have an implementation in the class where they are declared; their implementation is expected to be provided by concrete subclasses.
It would be nice if Rope emitted a warning.
Steps to reproduce the behavior:
- Code before refactoring:
from abc import abstractmethod
class Base:
@abstractmethod
def tag(self, text, tokenize=True):
return
- Apply the Inline Method refactoring to Base.tag().
- Langage dominant
- Python
- Étoiles
- 2.2k
- Forks
- 197
- Merge moyen
- 2 j 13 h
- PR mergées (30 j)
- 5
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de python-rope/rope
-
Difficulté 1/5 1-3 heures Accessibilité débutants 88/100
python-rope/rope#927 ·
Les mainteneurs répondent en général sous 7 jours
-
Extract raises IndexError or ModuleSyntaxError instead of RefactoringError for an empty or whitespace-only regionPeut-être pris @aliassar l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
python-rope/rope#926 ·
Les mainteneurs répondent en général sous 7 jours
-
simplify.real_code blanks t-strings, so renames inside them edit the wrong offsetPeut-être pris @sean0407 l’a pris il y a 10 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 83/100
python-rope/rope#913 ·
Les mainteneurs répondent en général sous 7 jours
-
Difficulté 4/5 3-5 jours Accessibilité débutants 30/100
python-rope/rope#925 ·
Les mainteneurs répondent en général sous 7 jours
-
Extract method/variable and introduce parameter don't check whether the new name is already in useOuverte
Difficulté 3/5 1-2 jours Accessibilité débutants 62/100
python-rope/rope#924 ·
Les mainteneurs répondent en général sous 7 jours
Toutes les issues de python-rope/rope
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
unionai-oss/pandera#2640 ·
Les mainteneurs répondent en général sous 2 jours
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
JdeRobot/RoboticsInfrastructure#843 ·
Les mainteneurs répondent en général sous 1 jour
-
automated endpoint-dead needs-curation
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
almanac-data/climate-almanac#122 ·
Les mainteneurs répondent en général sous 1 jour
-
area:harness enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
evarness-ai/iris-harness#420 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
suitenumerique/find#206 ·
Les mainteneurs répondent en général sous 2 jours