-
-
Notifications
You must be signed in to change notification settings - Fork 399
Removed System-Localization package #18121
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: Pharo14
Are you sure you want to change the base?
Removed System-Localization package #18121
Conversation
|
Tx seb good idea. |
|
Failing because I need to add the Writing to not forget |
|
The packages were not removed from the baseline so the bootstrap is failing |
| Extension { #name : 'String' } | ||
|
|
||
| { #category : '*System-Localization' } | ||
| String >> translated [ | ||
| "answer the receiver translated to the default language" | ||
| ^ NaturalLanguageTranslator translate: self | ||
| ] | ||
|
|
||
| { #category : '*System-Localization' } | ||
| String >> translatedTo: localeID [ | ||
| "answer the receiver translated to the given locale id" | ||
| ^ NaturalLanguageTranslator translate: self toLocale: localeID | ||
| ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe these extensions should be kept as deprecated just returning self for people using them
Fixes #17893
I propose to delete the package since the
LocalePluginis not present anymore in the vm. So, the whole package does not work. The primitives always fail.