Changes

Improvements on StartUpPreferences menu

- Precision on define preference file: there was no distinction between general/version preferences folder
- Added the same behaviour to generate preference file in version folder
- When creating a file, now it inspects it right after and the user can edit/delete the file directly when generating it: for me it is very logic to do that

now checks if the file already exists, inform the user it true and inspect the file anyway

Replace the process browser

This change removes the old process browser and replace it by the new one made by Esteban. 

This depends on this PR: https://github.com/pharo-spec/NewTools/pull/1103

Remove call to the old process browser API

I just removed because the method calling it is dead and other people in the team think it should be removed

Prevent renaming the accessors when renaming an instance variable

Could break code potentially because it was removing previous accessors

Removed Shout presenter as it should belong to NewTools repo and not Pharo

Updated baseline

Rename DateAndTime>>#fromMethodTimeStamp: to #fromMethodTimestamp: and implement it correctly

added transcript workerProcess to knownProcesses in ReleaseTest

Make the timestamp format of method changes and class comment changes in the change file consistent with the timestamp format for snapshot and quit entries.

Adapt tests after changing the timestamp format

Adapt tests that with timestamps

adding a first list of authors.
Extracted using

git shortlog -sne > AUTHORS

And loading the package https://github.com/Ducasse/MyBecher  package AuthorSplitter

Fixing picture reference

Adding core classes

Fixing filename of inspector

Update ReleaseTest.class.st

try to fix the fact that when Transcript is non interactive, we do not have a workerProcess :P

oops, this is a symbol

Update Beacon.md

Fix bug when adding an extension method overriding a trait method

In case you have a class using a trait with a method #toto, if you add an extension method named #toto on this class, then the method is not considered as an extension.

I'm adding regression tests and fixing this problem in this change

[Bootstrap] Remove shrink.st

shrink.st is not used in the bootstrap and the code is out of date. It references classes or packages that do not exist anymore and is using deprecated code.

I propose to remove it because it is not maintained

Remove the last references to RPackage

- Update comments
- Rename variables
- Rename methods
- Rename classes
- Update SUnitVisitor
- Introduce Deprecated14

Move some AST visitor methods to AST-Core

We have multiple #acceptVisitor:node: methods in Kernel-CodeModel. This methods are part of an AST visitor. 
I propose to package them in AST-Core instead of the Kernel

Improve #asMutator to raise an error on binary selectors

Fixes #4599

Remove dead code in Morphic

We have some code in Morphic that is only called by tests and this code is broken in most cases. 
Since Mophic should be removed in the future I propose to remove it instead of fixing it?
Fixes #18130

Simplify some Calypso code

Calypso has some code that cannot be reached in the current state of things.
Since we want to migrate this tool, I propose to remove it to simplify the code.

 Fixes #18033

[Refactorings] Clean test resource - remove RBBasicDummyLintRuleTest

RBBasicDummyLintRuleTest is almost not used anymore and has a lot of duplicated code with another test resource. I propose to only keep the other one and remove this one

Add back the class for test

Add accessor

Improvements on StartUpPreferences menu

- Precision on define preference file: there was no distinction between general/version preferences folder
- Added the same behaviour to generate preference file in version folder
- When creating a file, now it inspects it right after and the user can edit/delete the file directly when generating it: for me it is very logic to do that

now checks if the file already exists, inform the user it true and inspect the file anyway

Replace the process browser

This change removes the old process browser and replace it by the new one made by Esteban. 

This depends on this PR: https://github.com/pharo-spec/NewTools/pull/1103

Remove call to the old process browser API

I just removed because the method calling it is dead and other people in the team think it should be removed

Prevent renaming the accessors when renaming an instance variable

Could break code potentially because it was removing previous accessors

Removed Shout presenter as it should belong to NewTools repo and not Pharo

Updated baseline

Rename DateAndTime>>#fromMethodTimeStamp: to #fromMethodTimestamp: and implement it correctly

added transcript workerProcess to knownProcesses in ReleaseTest

Make the timestamp format of method changes and class comment changes in the change file consistent with the timestamp format for snapshot and quit entries.

Adapt tests after changing the timestamp format

Adapt tests that with timestamps

adding a first list of authors.
Extracted using

git shortlog -sne > AUTHORS

And loading the package https://github.com/Ducasse/MyBecher  package AuthorSplitter

Fixing picture reference

Adding core classes

Fixing filename of inspector

Update ReleaseTest.class.st

try to fix the fact that when Transcript is non interactive, we do not have a workerProcess :P

oops, this is a symbol

Update Beacon.md

Fix bug when adding an extension method overriding a trait method

In case you have a class using a trait with a method #toto, if you add an extension method named #toto on this class, then the method is not considered as an extension.

I'm adding regression tests and fixing this problem in this change

[Bootstrap] Remove shrink.st

shrink.st is not used in the bootstrap and the code is out of date. It references classes or packages that do not exist anymore and is using deprecated code.

I propose to remove it because it is not maintained

Remove the last references to RPackage

- Update comments
- Rename variables
- Rename methods
- Rename classes
- Update SUnitVisitor
- Introduce Deprecated14

Move some AST visitor methods to AST-Core

We have multiple #acceptVisitor:node: methods in Kernel-CodeModel. This methods are part of an AST visitor. 
I propose to package them in AST-Core instead of the Kernel

Improve #asMutator to raise an error on binary selectors

Fixes #4599

Remove dead code in Morphic

We have some code in Morphic that is only called by tests and this code is broken in most cases. 
Since Mophic should be removed in the future I propose to remove it instead of fixing it?
Fixes #18130

Simplify some Calypso code

Calypso has some code that cannot be reached in the current state of things.
Since we want to migrate this tool, I propose to remove it to simplify the code.

 Fixes #18033

[Refactorings] Clean test resource - remove RBBasicDummyLintRuleTest

RBBasicDummyLintRuleTest is almost not used anymore and has a lot of duplicated code with another test resource. I propose to only keep the other one and remove this one

Add back the class for test

Add accessor

Improvements on StartUpPreferences menu

- Precision on define preference file: there was no distinction between general/version preferences folder
- Added the same behaviour to generate preference file in version folder
- When creating a file, now it inspects it right after and the user can edit/delete the file directly when generating it: for me it is very logic to do that

now checks if the file already exists, inform the user it true and inspect the file anyway

Replace the process browser

This change removes the old process browser and replace it by the new one made by Esteban. 

This depends on this PR: https://github.com/pharo-spec/NewTools/pull/1103

Remove call to the old process browser API

I just removed because the method calling it is dead and other people in the team think it should be removed

Prevent renaming the accessors when renaming an instance variable

Could break code potentially because it was removing previous accessors

Removed Shout presenter as it should belong to NewTools repo and not Pharo

Updated baseline

Rename DateAndTime>>#fromMethodTimeStamp: to #fromMethodTimestamp: and implement it correctly

added transcript workerProcess to knownProcesses in ReleaseTest

Make the timestamp format of method changes and class comment changes in the change file consistent with the timestamp format for snapshot and quit entries.

Adapt tests after changing the timestamp format

Adapt tests that with timestamps

adding a first list of authors.
Extracted using

git shortlog -sne > AUTHORS

And loading the package https://github.com/Ducasse/MyBecher  package AuthorSplitter

Fixing picture reference

Adding core classes

Fixing filename of inspector

Update ReleaseTest.class.st

try to fix the fact that when Transcript is non interactive, we do not have a workerProcess :P

oops, this is a symbol

Update Beacon.md

Fix bug when adding an extension method overriding a trait method

In case you have a class using a trait with a method #toto, if you add an extension method named #toto on this class, then the method is not considered as an extension.

I'm adding regression tests and fixing this problem in this change

[Bootstrap] Remove shrink.st

shrink.st is not used in the bootstrap and the code is out of date. It references classes or packages that do not exist anymore and is using deprecated code.

I propose to remove it because it is not maintained

Remove the last references to RPackage

- Update comments
- Rename variables
- Rename methods
- Rename classes
- Update SUnitVisitor
- Introduce Deprecated14

Move some AST visitor methods to AST-Core

We have multiple #acceptVisitor:node: methods in Kernel-CodeModel. This methods are part of an AST visitor. 
I propose to package them in AST-Core instead of the Kernel

Improve #asMutator to raise an error on binary selectors

Fixes #4599

Remove dead code in Morphic

We have some code in Morphic that is only called by tests and this code is broken in most cases. 
Since Mophic should be removed in the future I propose to remove it instead of fixing it?
Fixes #18130

Simplify some Calypso code

Calypso has some code that cannot be reached in the current state of things.
Since we want to migrate this tool, I propose to remove it to simplify the code.

 Fixes #18033

[Refactorings] Clean test resource - remove RBBasicDummyLintRuleTest

RBBasicDummyLintRuleTest is almost not used anymore and has a lot of duplicated code with another test resource. I propose to only keep the other one and remove this one

Add back the class for test

Add accessor