Hernán Morales Durand

Results 137 comments of Hernán Morales Durand

Beware that for some reason the newly created class is not present in the newClasses instance variable.

> @hernanmd I relaunched the build because there were refactoring failing tests > > ``` > [MacOSX64.Refactoring.Transformations.Tests.RBRemoveSenderMethodParametrizedTest.testRemoveSenderIntoCascadedMessage(#rbClass->RBRemoveSenderRefactoring #constructor->#remove:inMethod:forClass: #index->1)](https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%20and%20branch%20Pipeline/job/PR-16917/3/testReport/junit/MacOSX64.Refactoring.Transformations.Tests/RBRemoveSenderMethodParametrizedTest/osx_64___Tests_osx_64___testRemoveSenderIntoCascadedMessage__rbClass__RBRemoveSenderRefactoring__constructor___remove_inMethod_forClass___index__1_/) > [unix-64 / Tests-unix-64 / Unix64.Refactoring.Transformations.Tests.RBRemoveSenderMethodParametrizedTest.testRemoveSenderIntoCascadedMessage(#rbClass->RBRemoveSenderRefactoring #constructor->#remove:inMethod:forClass: #index->1)](https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%20and%20branch%20Pipeline/job/PR-16917/3/testReport/junit/Unix64.Refactoring.Transformations.Tests/RBRemoveSenderMethodParametrizedTest/unix_64___Tests_unix_64___testRemoveSenderIntoCascadedMessage__rbClass__RBRemoveSenderRefactoring__constructor___remove_inMethod_forClass___index__1_/) > [windows-64 /...

> Errors seems related This PR would need #16953 first

It is already available and populated when is opened through the menu: Library -> Keymap descriptions

Feel free to close it if the reply applies

Something like this badre: ```smalltalk validateClassName | nameToValidate | name ifNil: [ ^self ]. "I try to convert to symbol, if there is an error the next guard will catch...

If I understood the issue correctly, I couldn't reproduce it (maybe it is already solved?) Please see the video: https://github.com/pharo-project/pharo/assets/4825959/c9c4fa0f-179b-4ad9-85d8-948915326050