Rename Field refactoring allows you to change the method parameter, causing inconsistent override
Les mainteneurs répondent en général sous 7 jours
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 25/100
Piste de recherche
Start by reproducing the issue with the BaseTagger and PatternTagger snippet, then apply Rename Field to the 'text' parameter of PatternTagger.tag. The payload does not name the rope source files, so locate the rename-field refactoring code yourself. Done means rope either warns or refuses when the renamed parameter belongs to a method that overrides a base-class method, and the snippet shows that behavior.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Rename Field refactoring allows you to change the method parameter, causing inconsistent override
It would be nice if Rope sent an alert to prevent the user from changing the overridden method signature
- Steps to reproduce the behavior:
from abc import abstractmethod
class BaseTagger:
@abstractmethod
def tag(self, text, tokenize=True):
return
class PatternTagger(BaseTagger):
def tag(self, text, tokenize=True):
if not isinstance(text, str):
text = text.raw
return [text, tokenize]
- Apply the Rename Field refactoring with the new name 'tag' to the field 'PatternTagger.tag.text'
- 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 11 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
-
area:cli bug difficulty:1
Difficulté 1/5 1-3 heures Accessibilité débutants 87/100
MakazhanAlpamys/Soup#2491 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
microsoft/onnxscript#3155 ·
Les mainteneurs répondent en général sous 4 jours
-
IFEval nth-paragraph checker reads the wrong paragraph when a blank separator is presentPeut-être pris @chetana1410 l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
EleutherAI/lm-evaluation-harness#4530 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
Les mainteneurs répondent en général sous 1 jour
-
sv_SE vat_id() has 14 digits: the "16" organisation number prefix is keptPeut-être pris @kevin9327 l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Les mainteneurs répondent en général sous 19 jours