Hi Evelyn - I’m glad your making progress on the refactorings - they were very unloved - to the point I had mostly stopped using them as they were either broken or didn’t work right. Given refactoring was basically invented in Smalltalk we had got into a bad place in the Pharo IDE so I am so pleased to see this getting sorted - and hopefully we can get back to a position where ours work as well as those in IntelliJ or Eclipse.
On this note - I haven’t noticed whether there is some work to add decent keystroke shortcuts to the most important ones so that you do them quickly. I am equally hopeful … that its a bit easier for us to write some of our own as there are some higher level ones that I would like to try creating (but gave up on as it wasn’t particularly obvious how to do it). The mention of refactoring2 sounds promising.
Anyway - just wanted to add a big thumbs up.
Tim
On Mon, 11 Jan 2021, at 3:09 AM, EVELYN CUSI LOPEZ wrote:
Hello everyone.
Last week I did these tasks:
- Fix "replace senders of message by another message" refactoring, the modification is for replace in all classes or only in owner class, this can be scale to replace in determinated classes (
https://github.com/pharo-project/pharo/pull/8314).
- Divide coupled logic between commands, views and refactorings (to remove NautilusRefactoring class).
- Review how to fix Undo Refactoring issue.
Tasks for this week:
- Fix undo refactoring issue.
- Migrate extract method to refactoring2.
- Review how migrate all refactoring commands to commander2 now only works in source code context.
Cheers,
Evelyn
Locked
