Progress Report -> Refactoring Project - ( December 14 - 18 )

EC
EVELYN CUSI LOPEZ
Mon, Dec 21, 2020 4:34 AM

Hi all,

Last week I did these tasks:

  • Pair programming with Pablo, to understand how use commander2 with
    calypso. By the moment this only works in source code context, some
    similary methods can be created to use commander2 in class, variable and
    method context.
  • Fix duplicated method issue (
    https://github.com/pharo-project/pharo/pull/8147 )
  • Fix all abstract instance variables issue (
    https://github.com/pharo-project/pharo/pull/8138)
  • Continue fixing Extract method and ocurrences issues, I separated middle
    interactions with user (in Find and replace refactoring), to remove
    coupling between view and logic, but now I have some errors when I want use
    a composite refactoring.

Tasks for this week:

Cheers,
Evelyn

Hi all, Last week I did these tasks: - Pair programming with Pablo, to understand how use commander2 with calypso. By the moment this only works in source code context, some similary methods can be created to use commander2 in class, variable and method context. - Fix duplicated method issue ( https://github.com/pharo-project/pharo/pull/8147 ) - Fix all abstract instance variables issue ( https://github.com/pharo-project/pharo/pull/8138) - Continue fixing Extract method and ocurrences issues, I separated middle interactions with user (in Find and replace refactoring), to remove coupling between view and logic, but now I have some errors when I want use a composite refactoring. Tasks for this week: - Finish to fix Extract method issues, only logic part (removing views) - Migrate extract method to refactoring2. - Migrate command of extract method and occurrences to commander2 - Create extract setUp and tearDown refactoring only for TestCases - Fix undo refactoring issue (I already tried to correct, however I continue having some problems - https://github.com/pharo-project/pharo/compare/Pharo9.0...Lin777:FixUndoRefactoring ) - Add option to limit classes where we replace senders of method to other method (https://github.com/pharo-project/pharo/issues/8087). Cheers, Evelyn