Re: [Pharo-dev] Update 100 for Pharo5!

2015-06-10 Thread Sergio Fedi
YAY! On Wed, Jun 10, 2015 at 4:12 AM, Marcus Denker marcus.den...@inria.fr wrote: Hi, We have update 100 for Pharo5! The issue tracker lists 290 cases with milestone Pharo5 that are closed. Marcus

Re: [Pharo-dev] Refactoring File Package

2015-06-08 Thread Sergio Fedi
One of the things I really like about ENVY (as found in VA Smalltalk) is that packages are formally modelled and are distinct from method categories. (Another is that a method can have multiple categories.) And by formally modelling packages, one can easily include proper dependency

Re: [Pharo-dev] About category

2015-05-31 Thread Sergio Fedi
Does this mean that a class from a Package will be able to have multiple Tags? The first time I saw Packages + Tags I tried to use multiple tags: Example of my initial impression: *Class - Tags* Car - Model CarTest - Model, Tests CarFactory - Factories CarFactoryTest - Factories, Tests

Re: [Pharo-dev] LinkedIn presence

2015-05-28 Thread Sergio Fedi
Thank you Yuri!​

Re: [Pharo-dev] Beyond RPackages: RPackageTag and DynamicGroup

2015-05-28 Thread Sergio Fedi
Thanks for the feedback. (Thanks Nicolai for confirming that) I understand all of the arguments. Somethings to spur things up: An argument in favor of having both classes handle comments is that from a user perspective: DynamicGroup - It's good to know the criteria under which classes are

[Pharo-dev] LinkedIn presence

2015-05-27 Thread Sergio Fedi
Hi, does Pharo have a presence on LinkedIn? I want to indicate my (tiny) contribution to the Pharo project and I don't find it as an Organization.

[Pharo-dev] Beyond RPackages: RPackageTag and DynamicGroup

2015-05-27 Thread Sergio Fedi
Hi list! As we are adding comments to Packages in Pharo, we stumbled upon the fact that there are other creatures present in the package list. These are: RPackageTag, which models the tags under the package DynamicGroup, which model the dynamic categories of LastModifiedClasses,

Re: [Pharo-dev] Lectures in africa :)

2015-05-24 Thread Sergio Fedi
Awesome! On Sun, May 24, 2015 at 5:44 AM, Alexandre Bergel alexandre.ber...@me.com wrote: Excellent! Thanks to share this impressive effort with us cheers, Alexandre On May 24, 2015, at 7:47 AM, stepharo steph...@free.fr wrote: Hi guys I m back to europe after one week to

Re: [Pharo-dev] New Collections for Pharo

2015-05-20 Thread Sergio Fedi
Nice work! I frequently needed the MultiValueDictionary and had it implemented several times in the past. ​

Re: [Pharo-dev] about the donwload win.zip pharo 40

2015-05-20 Thread Sergio Fedi
Worked for me too.​

Re: [Pharo-dev] [Pharo 5] For testing: enabled AST based syntax higlighting

2015-05-20 Thread Sergio Fedi
So cool.​

Re: [Pharo-dev] Null Coalesce Operator?

2015-05-20 Thread Sergio Fedi
In my experience those situations end up been handled by collections, detect:ifNone: which even handles the case when all defaults are nil. But maybe that idiom is very common in another situations.

Re: [Pharo-dev] New Collections for Pharo

2015-05-20 Thread Sergio Fedi
I agree with Hernan. In the MultiValueDictionary sometimes it's necessary to parametrize the collection the value has. On Wed, May 20, 2015 at 8:30 PM, Hernán Morales Durand hernan.mora...@gmail.com wrote: Hello Alexander, For the MultiValueDictionary IMHO you can need to use other

Re: [Pharo-dev] Manifest class values - instance or class side

2015-05-19 Thread Sergio Fedi
Why scary? It seems that it was a part of the system not used. Maybe the scary part was to have something implemented that wasn't used?

Re: [Pharo-dev] Manifest class values - instance or class side

2015-05-19 Thread Sergio Fedi
Supposing we do that, that we put these tests in RPackageTests, What tests de we put in Manifest-Tests?

Re: [Pharo-dev] Critics Browser should diff formatted original

2015-05-19 Thread Sergio Fedi
I agree. ​

Re: [Pharo-dev] Default package comment

2015-05-18 Thread Sergio Fedi
Personally, I do not fear to change the comment implementation later. That's what Smalltalk is good at. +1. But the most difficult thing is not to change the implementation but all its usage. if people start to use a feature and you change the way to use it after, it is more difficult to

Re: [Pharo-dev] FogBuz - Parent Cases

2015-05-17 Thread Sergio Fedi
Great explanation, thanks Ben. ​Is this behaviour documented somewhere? I found these articles: http://www.fogcreek.com/fogbugz/docs/70/topics/basics/LinkingCases.html http://www.fogcreek.com/fogbugz/docs/70/topics/basics/Subcases.html But some behaviours you listed weren't documented there.

[Pharo-dev] FogBuz - Parent Cases

2015-05-17 Thread Sergio Fedi
What are Parent Cases? What problem do they address? When should they be used?

Re: [Pharo-dev] Manifest class values - instance or class side

2015-05-17 Thread Sergio Fedi
I want to add a test to ensure #packageManifest answers a Class. Since RPackage is in RPackageCore, but the #packageManifest method is in Manifest-Core I'm not sure whether: 1. I have to make the test in RPackageTest but make the test method from the Manifest-Core package 2. I have to

Re: [Pharo-dev] Manifest class values - instance or class side

2015-05-17 Thread Sergio Fedi
Case created: https://pharo.fogbugz.com/f/cases/15557/New-PackageManifests-should-be-classes-not-instances Case Resolved (Review needed)

Re: [Pharo-dev] Manifest class values - instance or class side

2015-05-17 Thread Sergio Fedi
Case created: https://pharo.fogbugz.com/f/cases/15557/New-PackageManifests-should-be-classes-not-instances

Re: [Pharo-dev] Usage of stream

2015-05-16 Thread Sergio Fedi
Well, when retrieving some bytes from the read stream you will have them, temporarily, in a collection (OrderedCollection or Array I assume) With that you can use the messages from those collections: 'a3a' copyReplaceAll: '3' with: 'R' ​Which gives 'aRa'

Re: [Pharo-dev] Manifest class values - instance or class side

2015-05-16 Thread Sergio Fedi
RPackagepackageManifest Perhaps it should be filed as a bug? Strange thing is, I don't find senders of this method.

Re: [Pharo-dev] Usage of stream

2015-05-16 Thread Sergio Fedi
Do you have to modify the file? Create a new file with the modifications? Or get the file in memory but modified? On Sat, May 16, 2015 at 6:05 AM, Vincent BLONDEAU vincent.blond...@polytech-lille.net wrote: Hi, You can do it by using on a subclass of WriteStream peek and nextPut: methods.

Re: [Pharo-dev] Usage of stream

2015-05-16 Thread Sergio Fedi
Maybe you should open a stream reading the original file the open a writing stream for the copy file And reading from one, writing on the other, replacing what you need in the intermediate collection of bytes/characters ​

Re: [Pharo-dev] Default package comment

2015-05-15 Thread Sergio Fedi
Ok, I came up with this default comment. Let's see what you guys think of it: *This is a default comment for a package.* *To create a new one follow these guidelines:* * - Make the first line a one line description of the package and its purpose.* * - Describe with a more detailed description

Re: [Pharo-dev] Manifest class values - instance or class side

2015-05-15 Thread Sergio Fedi
Having said that, maybe we should strive for consistency to facilitate the next iteration: The refactoring which will convert one-class per Package to one-object per Package. ​

Re: [Pharo-dev] Manifest class values - instance or class side

2015-05-15 Thread Sergio Fedi
Class side because we do not want to create instances of manifest. Great, we'll do it this way. Thanks!

Re: [Pharo-dev] Default package comment

2015-05-15 Thread Sergio Fedi
Personally, I do not fear to change the comment implementation later. That's what Smalltalk is good at. But that doesn't mean it's the right choice :) The thing is that we can go full speed with this and make comments for: oneLineSummary longDescription usageDescription preInstallComment

Re: [Pharo-dev] Manifest class values - instance or class side

2015-05-15 Thread Sergio Fedi
​What about this method? RPackagepackageManifest ^ (self classes detect: [ :each | each isManifest ] ifNone: [ TheManifestBuilder new createManifestNamed: name]) *new* Anyone know why does it create an instance of the Manifest instead of returning the class? (or why are we using instances of

[Pharo-dev] Default package comment

2015-05-14 Thread Sergio Fedi
Hi guys, we are adding comments to packages and we would like a feedback from the community. What do you suggest for the default package comment?

Re: [Pharo-dev] Package comments

2015-05-06 Thread Sergio Fedi
Does this mean that we should focus on adding the comments on this ManifestXXX class? (instead of other implementations)​

Re: [Pharo-dev] Package comments

2015-05-05 Thread Sergio Fedi
If you need help, or just a buddy to tag along I can work with you.​

[Pharo-dev] Case 7256 - TestRunner should not deal with UI

2015-05-04 Thread Sergio Fedi
I'm plannig to tackle this case https://pharo.fogbugz.com/f/cases/7256/TestRunner-should-not-deal-with-UI I am all in for decoupling the TestRunner UI from the functionality it provides. And even making a new object that both the CommandLineTestRunner and the TestRunner use. I wouldn't do that

Re: [Pharo-dev] Improving the About Text of Versionner

2015-05-04 Thread Sergio Fedi
Wow! Look what I stirred up! Ok, I *vaguely* understand the two configuration schemes. I do understand how I touched two projects that are mantained differently and the problems in that. I'm not sure what to learn from all of this (I know I didn't do anything wrong) About the design issues.

Re: [Pharo-dev] Package comments

2015-05-04 Thread Sergio Fedi
I know little about the subject but Packages have been until recently only Strings. Now they were reified as objects, but as far as I saw, these objects didn't have comments as a part of them. ​

Re: [Pharo-dev] Improving the About Text of Versionner

2015-05-03 Thread Sergio Fedi
Ok, I added the case, submited the Slice and Resolved the case. https://pharo.fogbugz.com/f/cases/15480/Improving-the-About-information-of-Versionner The final text is this: *Versionner is tool for creating configurations.* *A Configuration determines the dependencies between packages. More

Re: [Pharo-dev] Improving the About Text of Versionner

2015-05-03 Thread Sergio Fedi

[Pharo-dev] Beyond resolving a case

2015-05-03 Thread Sergio Fedi
How do we state and/or document that further work is needed after a Case? For example, some things I had in mind after finishing this case: *While adding the About comment for the Versionner tool I've found that the About should be modeled (at least to encapsulate the about text and the about

Re: [Pharo-dev] Where to start contributing

2015-05-03 Thread Sergio Fedi
I saw that discussion, but I didn't know how I could (if I could) contribute to that problem. ​

Re: [Pharo-dev] Improving the About Text of Versionner

2015-05-03 Thread Sergio Fedi
There is ConfigurationOfVersionner in the image... Sorry, I don't understand. What does it mean? Is there something wrong I did?

Re: [Pharo-dev] Pharo Inbox Link pointing to Pharo40 Inbox still

2015-05-03 Thread Sergio Fedi
Ah, that's why I commited my changes to version 4 On Sun, May 3, 2015 at 6:26 AM, Norbert Hartl norb...@hartl.name wrote: On http://pharo.org/contribute-propose-fix in Prerequisites the Pharo Inbox link points to the pharo40inbox instead of pharo50inbox. Norbert

[Pharo-dev] Improving the About Text of Versionner

2015-05-02 Thread Sergio Fedi
This seemed a fairly easy change so I can keep honing my contributing skills. I came up with this About text for the tool. Below is a screenshot on how it looks: ​ Here's the text: *Versionner is a GUI for Metacello that allows you to use a nice (but limited, as are most GUI tools) tool

[Pharo-dev] Where to start contributing

2015-05-02 Thread Sergio Fedi
As a fairly experienced Smalltalk programmer but a very new user of Pharo, Where do you guys suggest me to start contributing? To be more specific, there are parts of the system I don't know, and practices and conventions of Pharo that I don't grasp yet, so it seems that work that can be done

Re: [Pharo-dev] Are these changes good enough

2015-05-01 Thread Sergio Fedi
Yes In the site it says that it is preferable via surface mail. But a fax is good enough. Does this still hold true? (if so, I would rather send it via fax, or scanned, or another medium)

Re: [Pharo-dev] Next Pharo sprint Moose dojo

2015-05-01 Thread Sergio Fedi
Couple of questions: 1 - Since this start at 5 am (Buenos Aires Time, GMT -3) until what hour is the sprint? 2 - Is this event appropiate/useful for newbies to Pharo?

Re: [Pharo-dev] Rectangle Center

2015-04-29 Thread Sergio Fedi
It seems that the problem is not with the Rectangle per se, but the type of measure it uses. Maybe the responsibility of being always an Integer measure should be delegated to the measure itself. Like, instead of using numbers, you could use Measures (from Aconcagua) that are 100 pixels instead

Re: [Pharo-dev] Rectangle Center

2015-04-29 Thread Sergio Fedi
On Wed, Apr 29, 2015 at 5:42 PM, Yuriy Tymchuk yuriy.tymc...@me.com wrote: Ok, can we start form the test? Is it ok, that the test passes for rectangle with 10 @ 20 extent and fails for 15 @ 20 extent? I would make it test both coordinates. i.e. test for 15 @ 15 extent

Re: [Pharo-dev] Are these changes good enough

2015-04-27 Thread Sergio Fedi
Great, it seems I've managed to upload the fix. *The issue number* is 15417 *This is the result of the commit* Name: SUnit-UI-SergioFedi.111 Author: SergioFedi Time: 27 April 2015, 7:16:53.814736 pm UUID: cbc836f0-a6bf-4847-a3be-da8b2b67657a Ancestors: SUnit-UI-TheIntegrator.110 Added

Re: [Pharo-dev] Are these changes good enough

2015-04-27 Thread Sergio Fedi
Great!​ I suppose the bug system will inform me when you finally resolve the case, right?

Re: [Pharo-dev] Are these changes good enough

2015-04-27 Thread Sergio Fedi
One thing I previously missed: marking it as resolved (and in need for a peer review) Already done that at: https://pharo.fogbugz.com/f/cases/resolve/15417/TestRunner-does-not-offer-the-RunCoverage-functionality ​

Re: [Pharo-dev] Are these changes good enough

2015-04-27 Thread Sergio Fedi
Oh, great, I misunderstood. Glad you clarified that. I'm gonna do it now. ​

Re: [Pharo-dev] Are these changes good enough

2015-04-27 Thread Sergio Fedi
Uploaded and marked as solved. ​Let's see how it goes.

[Pharo-dev] Are these changes good enough

2015-04-27 Thread Sergio Fedi
Hi, I'm new to the Pharo community and recently installed Pharo 4.0. While using it I've found that the TestRunner didn't have coverage and after investigating a little I've found that is was almost there and I made it work. Is this change something I could submit as a Fix? (as per in

Re: [Pharo-dev] Are these changes good enough

2015-04-27 Thread Sergio Fedi
I was following that video. And now I'm pretty sure my blunder was caused because I kept the package selected after revieweing the Changes. So instead of aving the Slice, I saved only the package. ​Thanks for the patience :)