Skip to content

Conversation

@guillep
Copy link
Member

@guillep guillep commented Aug 29, 2025

Many extesions are:

  • tests extending existing test classes that can be either integrated in the extended class or put in their own class
  • extensions that could be integrated in the extended class but maybe were not because of historical reasons
  • extensions that are used in a single place, have only that single use case, and can just be inlined in the caller

This PR solves ~300 extensions that look unjustified out of ~5K using the simple heuristics above:

  • move to the extended class
  • inline in caller
  • put in their own class

@guillep guillep merged commit 1fdf552 into pharo-project:Pharo14 Sep 3, 2025
1 check failed
@guillep guillep deleted the fix_extensions branch September 3, 2025 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant