Progress Report -> Refactoring Project - ( April 12 - April 16)

EC
Evelyn Cusi Lopez
Mon, Apr 19, 2021 5:08 AM

Hi everyone,

Last week I did these tasks:

Tasks for this week:

Regards,

Evelyn C.

Hi everyone, Last week I did these tasks: - *Improve and add command to deprecate a method. *Check PR <https://github.com/pharo-project/pharo/pull/9085> to know the changes. - *Improve "Set up score command" *Check PR <https://github.com/pharo-project/pharo/pull/9039> to know the changes. I'm not sure if this issue is still relevant, but you can see the new behavior here: https://youtu.be/ejL_RTLPOMo . -* Fix move method with suggestion of protocol* (Move to package of defining class enh #8929 <https://github.com/pharo-project/pharo/issues/8929>) Check PR <https://github.com/pharo-project/pharo/pull/9095> to know the changes. You can see the new behavior in this video: https://youtu.be/r22E1BUZxrE . - *Add preview rename class refactoring*, now show changes one by one, not all in one. Check PR <https://github.com/pharo-project/pharo/pull/9043> to know the changes. -* Introduce class deprecation refactoring*, It was based mainly on Stef's suggestions, however before doing the PR I hope to resolve some doubts to see if any of the behaviors from the previous refactoring should be preserved. You can see the changes here: https://github.com/pharo-project/pharo/compare/Pharo9.0...Lin777:ClassDeprecated . - I start to do the *new preview to set the default scope*. Still in progress. - I continue fixing the issue: RBNamespace doesn't support alias #9041 <https://github.com/pharo-project/pharo/issues/9041> and Inline method refactoring doesn't handle well, handling calls to super or overrides #9042 <https://github.com/pharo-project/pharo/issues/9042> . Still in progress. Tasks for this week: - Add command to class deprecation refactoring. - Finish to add the new preview to select a default scope. - Finish to fix: RBNamespace doesn't support alias #9041 <https://github.com/pharo-project/pharo/issues/9041> - Finish to fix issues: Inline method refactoring doesn't handle well, handling calls to super or overrides #9042 <https://github.com/pharo-project/pharo/issues/9042> and Inline senders refactoring does not correctly manage errors #6218 <https://github.com/pharo-project/pharo/issues/6218> - Fix issues: Enhance Refactoring Preview Widget #9060 <https://github.com/pharo-project/pharo/issues/9060> and Refactoring dialogs should not be modal #5894 . <https://github.com/pharo-project/pharo/issues/5894> <https://github.com/pharo-project/pharo/issues/6218> Regards, Evelyn C.
NB
Noury Bouraqadi
Mon, Apr 19, 2021 2:02 PM

Hi Evelyn,

I've been silently watching your weekly progress.
I thought, for once say thank you :-)

Noury
On Apr 19 2021, at 7:08 am, Evelyn Cusi Lopez cusi.evelyn@gmail.com wrote:

Hi everyone,

Last week I did these tasks:

Tasks for this week:

Refactoring dialogs should not be modal #5894 . (https://github.com/pharo-project/pharo/issues/5894)

Regards,

Evelyn C.

Hi Evelyn, I've been silently watching your weekly progress. I thought, for once say thank you :-) Noury On Apr 19 2021, at 7:08 am, Evelyn Cusi Lopez <cusi.evelyn@gmail.com> wrote: > Hi everyone, > > Last week I did these tasks: > > - Improve and add command to deprecate a method. Check PR (https://github.com/pharo-project/pharo/pull/9085) to know the changes. > - Improve "Set up score command" Check PR (https://github.com/pharo-project/pharo/pull/9039) to know the changes. I'm not sure if this issue is still relevant, but you can see the new behavior here: https://youtu.be/ejL_RTLPOMo . > > - Fix move method with suggestion of protocol (Move to package of defining class enh #8929 (https://github.com/pharo-project/pharo/issues/8929)) Check PR (https://github.com/pharo-project/pharo/pull/9095) to know the changes. You can see the new behavior in this video: https://youtu.be/r22E1BUZxrE . > - Add preview rename class refactoring, now show changes one by one, not all in one. Check PR (https://github.com/pharo-project/pharo/pull/9043) to know the changes. > > - Introduce class deprecation refactoring, It was based mainly on Stef's suggestions, however before doing the PR I hope to resolve some doubts to see if any of the behaviors from the previous refactoring should be preserved. You can see the changes here: https://github.com/pharo-project/pharo/compare/Pharo9.0...Lin777:ClassDeprecated . > - I start to do the new preview to set the default scope. Still in progress. > > - I continue fixing the issue: RBNamespace doesn't support alias #9041 (https://github.com/pharo-project/pharo/issues/9041) and Inline method refactoring doesn't handle well, handling calls to super or overrides #9042 (https://github.com/pharo-project/pharo/issues/9042) . Still in progress. > > Tasks for this week: > > - Add command to class deprecation refactoring. > - Finish to add the new preview to select a default scope. > > - Finish to fix: RBNamespace doesn't support alias #9041 (https://github.com/pharo-project/pharo/issues/9041) > - Finish to fix issues: Inline method refactoring doesn't handle well, handling calls to super or overrides #9042 (https://github.com/pharo-project/pharo/issues/9042) and Inline senders refactoring does not correctly manage errors #6218 (https://github.com/pharo-project/pharo/issues/6218) > - Fix issues: Enhance Refactoring Preview Widget #9060 (https://github.com/pharo-project/pharo/issues/9060) and > > Refactoring dialogs should not be modal #5894 . (https://github.com/pharo-project/pharo/issues/5894) > > > > > Regards, > > Evelyn C.
EC
Evelyn Cusi Lopez
Mon, Apr 19, 2021 2:24 PM

Hi Noury,

I  hope that the work we are doing with Stef, Pablo and Juan Pablo helps to
improve the user experience in the automatic refactorings of Pharo.
Any comments on the changes or the introduction of new ideas are welcome.

Regards,

Evelyn

El lun, 19 abr 2021 a las 10:03, Noury Bouraqadi (bouraqadi@gmail.com)
escribió:

Hi Evelyn,

I've been silently watching your weekly progress.
I thought, for once say thank you :-)

Noury
On Apr 19 2021, at 7:08 am, Evelyn Cusi Lopez cusi.evelyn@gmail.com
wrote:

Hi everyone,

Last week I did these tasks:

Tasks for this week:

Refactoring dialogs should not be modal #5894 .
https://github.com/pharo-project/pharo/issues/5894

Regards,

Evelyn C.

Hi Noury, I hope that the work we are doing with Stef, Pablo and Juan Pablo helps to improve the user experience in the automatic refactorings of Pharo. Any comments on the changes or the introduction of new ideas are welcome. Regards, Evelyn El lun, 19 abr 2021 a las 10:03, Noury Bouraqadi (<bouraqadi@gmail.com>) escribió: > Hi Evelyn, > > I've been silently watching your weekly progress. > I thought, for once say thank you :-) > > Noury > On Apr 19 2021, at 7:08 am, Evelyn Cusi Lopez <cusi.evelyn@gmail.com> > wrote: > > Hi everyone, > > Last week I did these tasks: > > - *Improve and add command to deprecate a method. *Check PR > <https://github.com/pharo-project/pharo/pull/9085> to know the changes. > - *Improve "Set up score command" *Check PR > <https://github.com/pharo-project/pharo/pull/9039> to know the changes. > I'm not sure if this issue is still relevant, but you can see the new > behavior here: https://youtu.be/ejL_RTLPOMo . > -* Fix move method with suggestion of protocol* (Move to package of > defining class enh #8929 > <https://github.com/pharo-project/pharo/issues/8929>) Check PR > <https://github.com/pharo-project/pharo/pull/9095> to know the changes. > You can see the new behavior in this video: https://youtu.be/r22E1BUZxrE . > - *Add preview rename class refactoring*, now show changes one by one, > not all in one. Check PR > <https://github.com/pharo-project/pharo/pull/9043> to know the changes. > -* Introduce class deprecation refactoring*, It was based mainly on > Stef's suggestions, however before doing the PR I hope to resolve some > doubts to see if any of the behaviors from the previous refactoring should > be preserved. You can see the changes here: > https://github.com/pharo-project/pharo/compare/Pharo9.0...Lin777:ClassDeprecated > . > - I start to do the *new preview to set the default scope*. Still in > progress. > - I continue fixing the issue: RBNamespace doesn't support alias #9041 > <https://github.com/pharo-project/pharo/issues/9041> and Inline method > refactoring doesn't handle well, handling calls to super or overrides #9042 > <https://github.com/pharo-project/pharo/issues/9042> . Still in progress. > > Tasks for this week: > > - Add command to class deprecation refactoring. > - Finish to add the new preview to select a default scope. > - Finish to fix: RBNamespace doesn't support alias #9041 > <https://github.com/pharo-project/pharo/issues/9041> > - Finish to fix issues: Inline method refactoring doesn't handle well, > handling calls to super or overrides #9042 > <https://github.com/pharo-project/pharo/issues/9042> and Inline senders > refactoring does not correctly manage errors #6218 > <https://github.com/pharo-project/pharo/issues/6218> > - Fix issues: Enhance Refactoring Preview Widget #9060 > <https://github.com/pharo-project/pharo/issues/9060> and > > Refactoring dialogs should not be modal #5894 . > <https://github.com/pharo-project/pharo/issues/5894> > > > > > Regards, > > Evelyn C. > >