Comments (4)
Because I did not want to override the existing methods and functionality if it is already present.
Now that Squeak 6.0 is here I could reconsider.
from squot.
Indeed, only FileSystem-Git code uses things from SquotCompatibility-SqueakCommon. With the old name Pharo-Compatbility it did not matter which code used it. :-) So I could rename that package.
But for the stub methods for read-only objects I would anyway need another compat package because the methods are available in Squeak 6.0, and the SquotCompatibility-Squeak50 has indeed methods used by Squot, not by FileSystem-Git.
from squot.
But now you could put the extension methods into the right compatibility package, couldn't you? Wouldn't this allow for clearer tracking of the origin of these overrides? Same for 21e8664#diff-e8aa5f417f3a14ad795b10c7c80746624d2c2a785cd04e3f5d5feab8b698cfe.
from squot.
Since this is in FileSystem-Git, not in Squot, I would have to create compatibility packages there in the first place. But wait, maybe the SquotCompatibility-SqueakCommon package is actually mislabelled now, since I certainly did not rely on Pharo methods in the "new" code (as compared to the "old" code in FileSystem-Git and FileSystem)!
from squot.
Related Issues (20)
- Key not found: objectClassName HOT 3
- Cannot add a package that consists only of extension methods without creating a class category for it
- .package confusion after setup HOT 1
- Allow to load author settings from ~/.gitconfig
- Removing a remote-tracking branch should ask whether to also delete it from the remote
- Cannot view a branch named "hooks" in the Git Browser
- Switching away from an empty branch errors HOT 3
- Git Browser cannot find my package to load/trick HOT 2
- Status indicators for branches in Git Browser HOT 4
- Remove FileList2 dependency
- Add instructions for citation HOT 3
- Redundant override of `PositionableStream>>#peek:`
- Dependency on old version of INIFile
- Squit: Highlight extension classes and methods in diff trees
- test
- Put the getting started text from the readme into a HelpBrowser page HOT 3
- Attempting to load Seaside fails HOT 2
- MessageNotUnderstood: UndefinedObject>>key from GitRefCache>>#allWithPrefix:ifAbsentPutAll: HOT 3
- Unpack git objects during pull in background? HOT 3
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from squot.