pharo-spec / NewTools

All development tools for Pharo, developed with Spec
21 stars 53 forks source link

Add StProtocolNameChooserPresenter #759

Closed jecisc closed 3 months ago

jecisc commented 4 months ago

This change introduces a class StProtocolNameChooserPresenter.

This new interface in Spec allows one to select a protocol name to replace the UI in morphic. Features:

I also propose to merge NewTools-Fuel, NewTools-Compression-Utils and this new presenter in a package NewTools-Utils to avoid having a lot of packages with only one class or one extension method

jecisc commented 4 months ago

I also added a bunch of tests

jecisc commented 4 months ago

Failing tests are unrelated

Ducasse commented 4 months ago

But why merging with fuel? I do not get the idea.

jecisc commented 4 months ago

It's to avoid to have a lot of packages with 1 class or 2 extension methods.

We had Compression-Utils with 2 extension methods and NewTools-Fuel with 2 extensions and 1 class. And here with the protocol chooser it would be yet another package with 1 class

Ducasse commented 4 months ago

Ok but to me the protocol chooser should be packaged with core functionality and zip and fuel not. If we start to group random things then we will get packages with strange dependencies. To me it is not awkward to have tiny packages. So I would package the chooser in a core package and not have a package for it.

jecisc commented 4 months ago

I moved it