StéphaneDucasse comments

Results 1102 comments of


                                            StéphaneDucasse

[Refactoring-UI] Error when pushing up method if same one exists in superclass

Thanks for the report

[Refactoring-UI] Error when pushing up method if same one exists in superclass

I can reproduce it.

Discard all when I have zero metadata raises an arror

We can reproduce it Pharo 13 (Pharo-13.0.0+SNAPSHOT.build.297.sha.9d19b62922b7fc72f44387254700dafd12c8c4a3 )

Discard all when I have zero metadata raises an arror

We defined ``` IceNoCommit >> shortId ^ 'nocommit id' ``` It does not raise an error anymore and the caller is the following one. ``` initializePresenters super initializePresenters. diffPanel leftLabel:...

Discard all when I have zero metadata raises an arror

@tesonep do you think that this is safe? I cannot simply run locally the tests

Push window does not honor selected remote

This is the same bug we faced with Quentin Moutté

Adapt contextmenu to actions

Thanks esteban. @koendehondt I will not integrate it immediately so that we can update the code in the code repo.

Adapt contextmenu to actions

Oki

Using new method extractInteger for parsing instead of asInteger

I have no idea how to fix the broken tests. I would prefer that Microdown is not used in Pharo. ``` ######################### # 2 tests did not pass: # #########################...

Using new method extractInteger for parsing instead of asInteger

Hi seb I will introduce asMicrodownInteger to simplify the dependency. Then once the PR is introduced in Pharo I will remove asMicrodownInteger.