NewTools icon indicating copy to clipboard operation
NewTools copied to clipboard
verified publisher icon pharo-spec

Removing dependency to UIManager in StDebugger

Open adri09070 opened this issue 1 year ago • 4 comments

Fixes https://github.com/pharo-project/pharo/issues/15430

Needs https://github.com/pharo-spec/Spec/pull/1540 to be merged first

adri09070 avatar May 02 '24 13:05 adri09070

Thanks adrian let us see what esteban will say. @estebanlm

Ducasse avatar May 04 '24 13:05 Ducasse

yes, this is fine but let's wait until the CI is working again (when Pharo13 is added)

estebanlm avatar May 06 '24 12:05 estebanlm

I added an issue to the main issue tracker for the CI:

https://github.com/pharo-project/pharo/issues/16589

MarcusDenker avatar May 07 '24 08:05 MarcusDenker

Test failure not related.

jecisc avatar May 16 '24 11:05 jecisc

@adri09070 Can you have a look at the conflict? I'm taking the lead to merge or close all the pending PRs because I'm fed up of the message we send around.

Ducasse avatar Jun 13 '24 07:06 Ducasse

This PR can simply be closed without merge. #760 completely removed the method that I changed and it's ok

adri09070 avatar Jun 13 '24 07:06 adri09070