[Pharo-project] SmallLint and Manifest
Hi, I have published a new version of the Critics Browser (menuWorld > Tool > CriticBrowser). The new version is - refactoring friendly. Yes we listened to you :). - the browser has been improved. It works with the latest Spec. With the Critics browser you can inspect violations of the SmallLint rules and mark them as false positive or true positive (ToDo) and see previous warnings flagged as false positives or toDo. In addition all the logic of the browser has been developed so that the manifest can be saved within the package without adding dependencies. In this new version the format of the manifest has changed to be compatible with tools refactoting, the rules are organized in a tree and itâs possible to adds comment when we mark a false positive or todo. We are writing a nice chapter on SmallLint, critics and other for the new book. Soon we will release it for reviews. Gofer new url: 'http://www.squeaksource.com/PharoTaskForces'; package: 'ConfigurationOfManifest'; load. ((Smalltalk at: #ConfigurationOfManifest) project version: #stable) load Simon
I presume that's 1.4 only? since I cannot load it in 2.0... On 2012-09-10, at 23:53, Simon Allier <simon.allier@inria.fr> wrote:
Hi,
I have published a new version of the Critics Browser (menuWorld > Tool > CriticBrowser). The new version is - refactoring friendly. Yes we listened to you :). - the browser has been improved. It works with the latest Spec.
With the Critics browser you can inspect violations of the SmallLint rules and mark them as false positive or true positive (ToDo) and see previous warnings flagged as false positives or toDo. In addition all the logic of the browser has been developed so that the manifest can be saved within the package without adding dependencies.
In this new version the format of the manifest has changed to be compatible with tools refactoting, the rules are organized in a tree and itâs possible to adds comment when we mark a false positive or todo.
We are writing a nice chapter on SmallLint, critics and other for the new book. Soon we will release it for reviews.
Gofer new url: 'http://www.squeaksource.com/PharoTaskForces'; package: 'ConfigurationOfManifest'; load. ((Smalltalk at: #ConfigurationOfManifest) project version: #stable) load
Simon
hey, I almost got it running :) - I got an error on ConfigurationOfRefactoringBrowser in the latest 2.0 image. However if I manually load all the packages from the squeaksource repos it works :) - Running all the Critics with all the classes in the Image raises an DNU on nil aContext protocols contains a nil entry cami
I presume that's 1.4 only? since I cannot load it in 2.0...
On 2012-09-10, at 23:53, Simon Allier <simon.allier@inria.fr> wrote:
Hi,
I have published a new version of the Critics Browser (menuWorld > Tool > CriticBrowser). The new version is - refactoring friendly. Yes we listened to you :). - the browser has been improved. It works with the latest Spec.
With the Critics browser you can inspect violations of the SmallLint rules and mark them as false positive or true positive (ToDo) and see previous warnings flagged as false positives or toDo. In addition all the logic of the browser has been developed so that the manifest can be saved within the package without adding dependencies.
In this new version the format of the manifest has changed to be compatible with tools refactoting, the rules are organized in a tree and itâs possible to adds comment when we mark a false positive or todo.
We are writing a nice chapter on SmallLint, critics and other for the new book. Soon we will release it for reviews.
Gofer new url: 'http://www.squeaksource.com/PharoTaskForces'; package: 'ConfigurationOfManifest'; load. ((Smalltalk at: #ConfigurationOfManifest) project version: #stable) load
Simon
Thank for the feedback. On Sep 11, 2012, at 12:06 PM, Camillo Bruni wrote:
hey, I almost got it running :)
- I got an error on ConfigurationOfRefactoringBrowser in the latest 2.0 image. However if I manually load all the packages from the squeaksource repos it works :)
I dont have any problem for loading the configuration in the last pharo 2.0.
- Running all the Critics with all the classes in the Image raises an DNU on nil aContext protocols contains a nil entry
cami
I presume that's 1.4 only? since I cannot load it in 2.0...
On 2012-09-10, at 23:53, Simon Allier <simon.allier@inria.fr> wrote:
Hi,
I have published a new version of the Critics Browser (menuWorld > Tool > CriticBrowser). The new version is - refactoring friendly. Yes we listened to you :). - the browser has been improved. It works with the latest Spec.
With the Critics browser you can inspect violations of the SmallLint rules and mark them as false positive or true positive (ToDo) and see previous warnings flagged as false positives or toDo. In addition all the logic of the browser has been developed so that the manifest can be saved within the package without adding dependencies.
In this new version the format of the manifest has changed to be compatible with tools refactoting, the rules are organized in a tree and itâs possible to adds comment when we mark a false positive or todo.
We are writing a nice chapter on SmallLint, critics and other for the new book. Soon we will release it for reviews.
Gofer new url: 'http://www.squeaksource.com/PharoTaskForces'; package: 'ConfigurationOfManifest'; load. ((Smalltalk at: #ConfigurationOfManifest) project version: #stable) load
Simon
On 2012-09-11, at 13:46, Simon Allier <simon.allier@inria.fr> wrote:
Thank for the feedback. On Sep 11, 2012, at 12:06 PM, Camillo Bruni wrote:
hey, I almost got it running :)
- I got an error on ConfigurationOfRefactoringBrowser in the latest 2.0 image. However if I manually load all the packages from the squeaksource repos it works :)
I dont have any problem for loading the configuration in the last pharo 2.0.
sorry, you're right must be some strange image setup of mine...
I've tried in a 1.4, and it does not work "SystemAnnouncer uniqueInstance weak" raises an error. What is the status of 2.0? Is it usable now? Cheers, Alexandre On Sep 11, 2012, at 6:24 AM, Camillo Bruni <camillobruni@gmail.com> wrote:
I presume that's 1.4 only? since I cannot load it in 2.0...
On 2012-09-10, at 23:53, Simon Allier <simon.allier@inria.fr> wrote:
Hi,
I have published a new version of the Critics Browser (menuWorld > Tool > CriticBrowser). The new version is - refactoring friendly. Yes we listened to you :). - the browser has been improved. It works with the latest Spec.
With the Critics browser you can inspect violations of the SmallLint rules and mark them as false positive or true positive (ToDo) and see previous warnings flagged as false positives or toDo. In addition all the logic of the browser has been developed so that the manifest can be saved within the package without adding dependencies.
In this new version the format of the manifest has changed to be compatible with tools refactoting, the rules are organized in a tree and itâs possible to adds comment when we mark a false positive or todo.
We are writing a nice chapter on SmallLint, critics and other for the new book. Soon we will release it for reviews.
Gofer new url: 'http://www.squeaksource.com/PharoTaskForces'; package: 'ConfigurationOfManifest'; load. ((Smalltalk at: #ConfigurationOfManifest) project version: #stable) load
Simon
-- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
On 2012-09-11, at 13:57, Alexandre Bergel <alexandre.bergel@me.com> wrote:
I've tried in a 1.4, and it does not work "SystemAnnouncer uniqueInstance weak" raises an error. What is the status of 2.0? Is it usable now?
nope, there are still nasty bugs with RPackage around, for instance: https://code.google.com/p/pharo/issues/detail?id=6648
On Mon, Sep 10, 2012 at 11:53 PM, Simon Allier <simon.allier@inria.fr>wrote:
Hi,
I have published a new version of the Critics Browser (menuWorld > Tool > CriticBrowser). The new version is - refactoring friendly. Yes we listened to you :).
hehehe I felt touched by that ;( Thanks Simon, these are really great news because it doesn't mean we have to throw the manifiest away after a couple of refactors :) I will re-run everything with Fuel soon and give you any feedback I have.
- the browser has been improved. It works with the latest Spec.
With the Critics browser you can inspect violations of the SmallLint rules and mark them as false positive or true positive (ToDo) and see previous warnings flagged as false positives or toDo. In addition all the logic of the browser has been developed so that the manifest can be saved within the package without adding dependencies.
In this new version the format of the manifest has changed to be compatible with tools refactoting, the rules are organized in a tree and itâs possible to adds comment when we mark a false positive or todo.
We are writing a nice chapter on SmallLint, critics and other for the new book. Soon we will release it for reviews.
Gofer new url: 'http://www.squeaksource.com/PharoTaskForces'; package: 'ConfigurationOfManifest'; load. ((Smalltalk at: #ConfigurationOfManifest) project version: #stable) load
Simon
-- Mariano http://marianopeck.wordpress.com
I have fix the error found by Camillo, now it works with all packages and rules in the last pharo 2.0 image. On Sep 11, 2012, at 4:13 PM, Mariano Martinez Peck wrote:
On Mon, Sep 10, 2012 at 11:53 PM, Simon Allier <simon.allier@inria.fr> wrote: Hi,
I have published a new version of the Critics Browser (menuWorld > Tool > CriticBrowser). The new version is - refactoring friendly. Yes we listened to you :).
hehehe I felt touched by that ;( Thanks Simon, these are really great news because it doesn't mean we have to throw the manifiest away after a couple of refactors :) I will re-run everything with Fuel soon and give you any feedback I have.
- the browser has been improved. It works with the latest Spec.
With the Critics browser you can inspect violations of the SmallLint rules and mark them as false positive or true positive (ToDo) and see previous warnings flagged as false positives or toDo. In addition all the logic of the browser has been developed so that the manifest can be saved within the package without adding dependencies.
In this new version the format of the manifest has changed to be compatible with tools refactoting, the rules are organized in a tree and itâs possible to adds comment when we mark a false positive or todo.
We are writing a nice chapter on SmallLint, critics and other for the new book. Soon we will release it for reviews.
Gofer new url: 'http://www.squeaksource.com/PharoTaskForces'; package: 'ConfigurationOfManifest'; load. ((Smalltalk at: #ConfigurationOfManifest) project version: #stable) load
Simon
-- Mariano http://marianopeck.wordpress.com
excellent! Simon what about producing a video so that the world can see your nice tool in action. Stef On Sep 11, 2012, at 4:21 PM, Simon Allier wrote:
I have fix the error found by Camillo, now it works with all packages and rules in the last pharo 2.0 image.
On Sep 11, 2012, at 4:13 PM, Mariano Martinez Peck wrote:
On Mon, Sep 10, 2012 at 11:53 PM, Simon Allier <simon.allier@inria.fr> wrote: Hi,
I have published a new version of the Critics Browser (menuWorld > Tool > CriticBrowser). The new version is - refactoring friendly. Yes we listened to you :).
hehehe I felt touched by that ;( Thanks Simon, these are really great news because it doesn't mean we have to throw the manifiest away after a couple of refactors :) I will re-run everything with Fuel soon and give you any feedback I have.
- the browser has been improved. It works with the latest Spec.
With the Critics browser you can inspect violations of the SmallLint rules and mark them as false positive or true positive (ToDo) and see previous warnings flagged as false positives or toDo. In addition all the logic of the browser has been developed so that the manifest can be saved within the package without adding dependencies.
In this new version the format of the manifest has changed to be compatible with tools refactoting, the rules are organized in a tree and itâs possible to adds comment when we mark a false positive or todo.
We are writing a nice chapter on SmallLint, critics and other for the new book. Soon we will release it for reviews.
Gofer new url: 'http://www.squeaksource.com/PharoTaskForces'; package: 'ConfigurationOfManifest'; load. ((Smalltalk at: #ConfigurationOfManifest) project version: #stable) load
Simon
-- Mariano http://marianopeck.wordpress.com
I started to use it, and I will have suggestion on the GUI. Cheers, Alexandre On Sep 11, 2012, at 1:52 PM, Stéphane Ducasse <stephane.ducasse@inria.fr> wrote:
excellent! Simon what about producing a video so that the world can see your nice tool in action.
Stef
On Sep 11, 2012, at 4:21 PM, Simon Allier wrote:
I have fix the error found by Camillo, now it works with all packages and rules in the last pharo 2.0 image.
On Sep 11, 2012, at 4:13 PM, Mariano Martinez Peck wrote:
On Mon, Sep 10, 2012 at 11:53 PM, Simon Allier <simon.allier@inria.fr> wrote: Hi,
I have published a new version of the Critics Browser (menuWorld > Tool > CriticBrowser). The new version is - refactoring friendly. Yes we listened to you :).
hehehe I felt touched by that ;( Thanks Simon, these are really great news because it doesn't mean we have to throw the manifiest away after a couple of refactors :) I will re-run everything with Fuel soon and give you any feedback I have.
- the browser has been improved. It works with the latest Spec.
With the Critics browser you can inspect violations of the SmallLint rules and mark them as false positive or true positive (ToDo) and see previous warnings flagged as false positives or toDo. In addition all the logic of the browser has been developed so that the manifest can be saved within the package without adding dependencies.
In this new version the format of the manifest has changed to be compatible with tools refactoting, the rules are organized in a tree and itâs possible to adds comment when we mark a false positive or todo.
We are writing a nice chapter on SmallLint, critics and other for the new book. Soon we will release it for reviews.
Gofer new url: 'http://www.squeaksource.com/PharoTaskForces'; package: 'ConfigurationOfManifest'; load. ((Smalltalk at: #ConfigurationOfManifest) project version: #stable) load
Simon
-- Mariano http://marianopeck.wordpress.com
-- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
Hi Simon. I have just run it in Tanker. It already help me to detect a couple of things :) Some feedback: 1) Is there a way to set the package of the Manifest class? Because I don't like the default package where it goes.... Now it is even worst because it goes to "UndefinedPackage": 2) I have a weird bug rendering some text. (look attached) Thanks! On Tue, Sep 11, 2012 at 7:19 PM, Alexandre Bergel <alexandre.bergel@me.com>wrote:
I started to use it, and I will have suggestion on the GUI.
Cheers, Alexandre
On Sep 11, 2012, at 1:52 PM, Stéphane Ducasse <stephane.ducasse@inria.fr> wrote:
excellent! Simon what about producing a video so that the world can see your nice tool in action.
Stef
On Sep 11, 2012, at 4:21 PM, Simon Allier wrote:
I have fix the error found by Camillo, now it works with all packages and rules in the last pharo 2.0 image.
On Sep 11, 2012, at 4:13 PM, Mariano Martinez Peck wrote:
On Mon, Sep 10, 2012 at 11:53 PM, Simon Allier <simon.allier@inria.fr>
wrote:
Hi,
I have published a new version of the Critics Browser (menuWorld > Tool > CriticBrowser). The new version is - refactoring friendly. Yes we listened to you :).
hehehe I felt touched by that ;( Thanks Simon, these are really great news because it doesn't mean we have to throw the manifiest away after a couple of refactors :) I will re-run everything with Fuel soon and give you any feedback I have.
- the browser has been improved. It works with the latest Spec.
With the Critics browser you can inspect violations of the SmallLint rules and mark them as false positive or true positive (ToDo) and see previous warnings flagged as false positives or toDo. In addition all the logic of the browser has been developed so that the manifest can be saved within the package without adding dependencies.
In this new version the format of the manifest has changed to be compatible with tools refactoting, the rules are organized in a tree and itâs possible to adds comment when we mark a false positive or todo.
We are writing a nice chapter on SmallLint, critics and other for the new book. Soon we will release it for reviews.
Gofer new url: 'http://www.squeaksource.com/PharoTaskForces'; package: 'ConfigurationOfManifest'; load. ((Smalltalk at: #ConfigurationOfManifest) project version: #stable) load
Simon
-- Mariano http://marianopeck.wordpress.com
-- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
-- Mariano http://marianopeck.wordpress.com
On Sep 11, 2012, at 10:44 PM, Mariano Martinez Peck wrote:
Hi Simon. I have just run it in Tanker. It already help me to detect a couple of things :) Some feedback:
1) Is there a way to set the package of the Manifest class?
normally it should always go to the package automatically.
Because I don't like the default package where it goes.... Now it is even worst because it goes to "UndefinedPackage":
this is probably because of the shaking of rpackage.
2) I have a weird bug rendering some text. (look attached)
Thanks!
On Tue, Sep 11, 2012 at 7:19 PM, Alexandre Bergel <alexandre.bergel@me.com> wrote: I started to use it, and I will have suggestion on the GUI.
Cheers, Alexandre
On Sep 11, 2012, at 1:52 PM, Stéphane Ducasse <stephane.ducasse@inria.fr> wrote:
excellent! Simon what about producing a video so that the world can see your nice tool in action.
Stef
On Sep 11, 2012, at 4:21 PM, Simon Allier wrote:
I have fix the error found by Camillo, now it works with all packages and rules in the last pharo 2.0 image.
On Sep 11, 2012, at 4:13 PM, Mariano Martinez Peck wrote:
On Mon, Sep 10, 2012 at 11:53 PM, Simon Allier <simon.allier@inria.fr> wrote: Hi,
I have published a new version of the Critics Browser (menuWorld > Tool > CriticBrowser). The new version is - refactoring friendly. Yes we listened to you :).
hehehe I felt touched by that ;( Thanks Simon, these are really great news because it doesn't mean we have to throw the manifiest away after a couple of refactors :) I will re-run everything with Fuel soon and give you any feedback I have.
- the browser has been improved. It works with the latest Spec.
With the Critics browser you can inspect violations of the SmallLint rules and mark them as false positive or true positive (ToDo) and see previous warnings flagged as false positives or toDo. In addition all the logic of the browser has been developed so that the manifest can be saved within the package without adding dependencies.
In this new version the format of the manifest has changed to be compatible with tools refactoting, the rules are organized in a tree and itâs possible to adds comment when we mark a false positive or todo.
We are writing a nice chapter on SmallLint, critics and other for the new book. Soon we will release it for reviews.
Gofer new url: 'http://www.squeaksource.com/PharoTaskForces'; package: 'ConfigurationOfManifest'; load. ((Smalltalk at: #ConfigurationOfManifest) project version: #stable) load
Simon
-- Mariano http://marianopeck.wordpress.com
-- _,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;: Alexandre Bergel http://www.bergel.eu ^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.
-- Mariano http://marianopeck.wordpress.com
<Screen shot 2012-09-11 at 10.18.11 PM.png>
participants (5)
-
Alexandre Bergel -
Camillo Bruni -
Mariano Martinez Peck -
Simon Allier -
Stéphane Ducasse