pharo icon indicating copy to clipboard operation
pharo copied to clipboard

Merge RBRenameTemporaryVariableTransformation and RBRenameTemporaryVariableRefactoring

Open Ducasse opened this issue 3 months ago • 3 comments

Since the refactoring version does the same and its implementation has been revisited. In addition there is not breakingChanges precondition. So ignoring them is not possible so it makes the transformation useless.

Ducasse avatar Nov 08 '25 16:11 Ducasse

@balsa-sarenac what do you think? I checked both implementations.

Ducasse avatar Nov 08 '25 16:11 Ducasse

I will try to get the variable API because the transformation is asking for a variable while the refactoring an interval.

Ducasse avatar Nov 08 '25 16:11 Ducasse

I agree. We can kill transformation and unify apis (maybe give both interval and variable). If I remember correctly, transformation accepts variable and then searches for it?

balsa-sarenac avatar Nov 10 '25 09:11 balsa-sarenac