search

squeak-smalltalk / squeak-ston

Fork of the STON project for Squeak/Smalltalk
MIT License
2 stars 2 forks source link

How to merge upstream #6

Open marceltaeumel opened 2 years ago

marceltaeumel commented 2 years ago

Here is a change-set that makes the current "master" in upstream compatible with Squeak6.0alpha.

fix-squeak-ston.5.cs.txt

marceltaeumel commented 2 years ago

Here is some extra documentation for the upstream maintainers:

Maybe you can infer a few new ideas on how to improve modularity and extension points upstream. For example, the use of STONZnUrl and STONFileReference in "STON-Core" seems unnecessary. Also their base class not being Object. The package "STON-Extensions" might be a better fit. Same for the extensions in SmallDictionary.

Since there is a Pharo-specific "STON-Extensions" package, it might be beneficial to mirror this structure in test packages. There is currently only "STON-Tests". A few tests might benefit from being extracted into a "STON-Tests-Extensions" for example. Then I could also add a "STON-Tests-ExtensionsForSqueak".

The rather small package "STON-Text support" could be merged into "STON-Core" or "STON-Extensions" as well.