Inline Method refactoring is allowed in methods of the descriptor protocol
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
The payload gives only the reproduction: Inline Method applied to __get__ in a descriptor class removes the method entirely. Start by reproducing that with the snippet in the Inline Method refactoring's tests, then find where the refactoring decides which methods it may inline. Done means either the descriptor method is left intact or the refactoring warns and declines, and the agreed behaviour is confirmed in the thread first.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Steps to reproduce the behavior:
- Code before refactoring:
class Descriptor:
def __get__(self, instance, owner):
return 40
-
Apply the Inline Method refactoring to the method __get__.
-
Expected code after refactoring: the same as the original. It would be nice if Rope could emit a warning that applying the refactoring to this type of class is not allowed.
-
Rope produces the result below:
class Descriptor:
pass
- 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
-
Backend issue lists report in-patch warnings and out-of-patch errors as not publishablePeut-être pris @RaphaelFakhri l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
mozilla/code-review#3794 ·
Les mainteneurs répondent en général sous 1 jour
-
[oblt-aw][autodoc] Update docs that still describe the security registry as emptyPeut-être pris @github-actions l’a pris aujourd’hui. Ouverte
Difficulté 1/5 1-3 heures Accessibilité débutants 85/100
Les mainteneurs répondent en général sous 1 jour
-
[Bug] adb backup script reports "Done" & exits 0 when the backup failsPeut-être pris @bhattmanthan l’a pris aujourd’hui. Ouverterequest-issue-assignment
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
OWASP/mastg#4034 · 5 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
agent-ui bug p2
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
amd/gaia#6580 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
correction metadata
Difficulté 1/5 Moins d'une heure Accessibilité débutants 72/100
acl-org/acl-anthology#10768 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour