Hi,

On 4 Feb 2019, at 05:29, Paul DeBruicker <pdebruic@gmail.com> wrote:

Thanks Cyril.

For those projects I don't own/maintain that are stored in MCZ files is there a way to see who authored the method? Or is Iceberg + git-blame the only way to go now?

Those will show the ts/author as before.

If git-blame is the only way is there a way to have all the repos an image uses brought under git so I can git-blame everything automatically?

Answered below :)

Without having to do anything by hand when a package is loaded, I mean. And also a way to see that info right next to the code pane without having to click anything? Also hopefully senders/implementors count?

Uhm? I do not understand this… but I think is not related to iceberg but calypso, isn’t? If is calypso, and what you ask is to have the references available (for example in another tab) this is not there but is easy to implement. In fact, I remember an iteration of calypso that had them, and I think the reason while is gone is that It becomes kind of a pain in large images (but now we have real lazy tabs so we could review that decision).

When I read code I skim code written by other people (especially people with more experience with the domain, like your MDL package) and assume it's more correct because they've published it. And more critically review what I've written in/against those packages.

Without the author and timestamp visible next to the method code pane it is harder to know whether I should trust or be more skeptical of the method.

In Pharo this was always after a click. It is still now always after a click.

I don't always remember which parts I wrote and which parts someone else wrote.

Same. A click will tell you :) Esteban

Paul

CyrilFerlicot wrote

...

On Sun 3 Feb 2019 at 22:02, PAUL DEBRUICKER <

...

pdebruic@

...

> wrote:

...

Hi -

How can I see who wrote a method and when they wrote it in Calypso?

Hello,

You need to add Calypso to Iceberg and have a locale clone linked. Then an « history » button will appear in Calypso whose info are extracted via git blame feature of git.

...

In Nautilus it was the AnnotationPane plugin IIRC.

Thanks

Paul

--

Cyril Ferlicot https://ferlicot.fr

-- Sent from: http://forum.world.st/Pharo-Smalltalk-Users-f1310670.html <http://forum.world.st/Pharo-Smalltalk-Users-f1310670.html>

On 4 Feb 2019, at 16:02, Paul DeBruicker <pdebruic@gmail.com> wrote:

Hi

EstebanLM wrote

...

...

Without having to do anything by hand when a package is loaded, I mean. And also a way to see that info right next to the code pane without having to click anything? Also hopefully senders/implementors count?

Uhm? I do not understand this… but I think is not related to iceberg but calypso, isn’t? If is calypso, and what you ask is to have the references available (for example in another tab) this is not there but is easy to implement. In fact, I remember an iteration of calypso that had them, and I think the reason while is gone is that It becomes kind of a pain in large images (but now we have real lazy tabs so we could review that decision).

Yeah you're right. Calypso. MY reading of Cyril's response made me think it was a calypso/iceberg integration I'd have to do/set-up manually for every package rather than something automatically handled when a package is loaded into the image.

In Nautilus there was the AnnotationPanePlugin that showed author/timestamp/sender count/implementer count without any intervention on my part. See attached from Pharo 6. Once you turned the AnnotationPanePlugin on in any nautilus browser it was on in all the browsers you create in the future. Omnibrowser did it automatically IIRC.

Just wondering if there is something like that already in Pharo 7 or if not how to get it in Calypso.

Ah yes, but it was not default nor widely used (in fact, you are the first one I understand it was using it). And no, there is nothing like that in Calypso for the moment (doable, but not there) Esteban

Paul

Nautilus_with_annotation_pane.png <http://forum.world.st/file/t94460/Nautilus_with_annotation_pane.png>

-- Sent from: http://forum.world.st/Pharo-Smalltalk-Users-f1310670.html