Cleanup old methods

[jbrichau]

• Removes old code mentioned in issue Fate of old compatibility methods  #1417
• Removes GRPackage extensions. This mechanism for declaring dependencies between packages pre-dates the Metacello configuration and baseline. It's no longer up to date and unused.
• Remove broken WADevelopment>>#cleanupCategories (see WADevelopment class >> #cleanupCategories likely broken on Pharo 10+ #1343). This can now be better handled via code critiques as well.

[codecov]

Codecov Report

Attention: Patch coverage is 1.97628% with 248 lines in your changes missing coverage. Please review.

Project coverage is 49.33%. Comparing base (cb4b8ef) to head (fde0005).

Files	Patch %	Lines
...deRemainingBecausePharo8LimitationOfLiteralsTo..st	0.00%	132 Missing ⚠️
...nalTest.class/instance/countriesPerCurrencyCode.st	0.00%	66 Missing ⚠️
...alTest.class/instance/currenciesAndCountriesOn..st	0.00%	10 Missing ⚠️
...eFunctionalTest.class/instance/countryNamesFor..st	0.00%	8 Missing ⚠️
...eFunctionalTest.class/instance/renderContentOn..st	0.00%	8 Missing ⚠️
...est.class/instance/renderSelectedCountryNameOn..st	0.00%	6 Missing ⚠️
...unctionalTest.class/instance/listOfCountryNames.st	0.00%	4 Missing ⚠️
...alTest.class/instance/countriesForCurrencyCode..st	0.00%	4 Missing ⚠️
...nctionalTest.class/instance/listOfCurrencyCodes.st	0.00%	4 Missing ⚠️
...pleteFunctionalTest.class/instance/countryNames.st	0.00%	4 Missing ⚠️
... and 2 more

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1435      +/-   ##
==========================================
+ Coverage   49.04%   49.33%   +0.29%     
==========================================
  Files        9143     9073      -70     
  Lines       80555    80055     -500     
==========================================
- Hits        39506    39498       -8     
+ Misses      41049    40557     -492     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[marschall]

Maybe I'm missing something here. Unfortunately we still have some senders of GRPackage class >> #grPackages

• JQUiFunctionalTest >> #packages
• JQFunctionalTest >> #seasidePackagesOn:
• JQFunctionalTest >> #seasidePackagesOn:
• WADevelopment class >> # graphFunctionalDependencies:tests:
• WADevelopment class >> #allGRPackages
  • WADevelopment class >> #allDirtyPackages
  • WADevelopment class >> #browserEnvironment
  • WADevelopment class >> #cleanupCategories
  • WADevelopment class >> #gofers
  • WADevelopment class >> #graphFunctionalDependencies:tests:

[jbrichau]

Yes, that was not so good... just did not check those.

• In the functional tests, I replaced its use with something completely different (country names and currencies) to ensure these are not dependent on a specific platform.
• In WADevelopment, I removed the methods that referred it. None of these have been in use for the last 10 years and are replaced by either Metacello or code critiques in Pharo.

[marschall]

LGTM