Progress Report -> Refactoring Project - ( January 4 - 8 )

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

Progress Report -> Refactoring Project - ( January 4 - 8 )

EVELYN CUSI LOPEZ
Hello everyone.

Last week I did these tasks:

- Fix PR of "extract method and occurrences" refactoring (https://github.com/pharo-project/pharo/pull/8193).
- 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.
- Fix rename protocol issue (https://github.com/pharo-project/pharo/issues/4461).
- Fix add argument issue (https://github.com/pharo-project/pharo/issues/5852).

Cheers,
Evelyn
Reply | Threaded
Open this post in threaded view
|

Re: Progress Report -> Refactoring Project - ( January 4 - 8 )

Tim Mackinnon
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 PR of "extract method and occurrences" refactoring (https://github.com/pharo-project/pharo/pull/8193).
- 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

Reply | Threaded
Open this post in threaded view
|

Re: Progress Report -> Refactoring Project - ( January 4 - 8 )

EVELYN CUSI LOPEZ
Hi Tim,

Thanks, we are working to have an useful easy to use refactoring tool. Any comment to improve is welcome ☺

El lun, 11 ene 2021 a las 5:23, Tim Mackinnon (<[hidden email]>) escribió:
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 PR of "extract method and occurrences" refactoring (https://github.com/pharo-project/pharo/pull/8193).
- 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