Le 21/05/2017 à 05:12, Ben Coman a écrit : > Is this of concern for Pharo...? "Pull requests on GitHub where the > Trait comments get removed (because they were not loaded with the > package in the first place)." > > http://forum.world.st/Re-The-Inbox-System-jr-919-mcz-td4941798.html#none > > http://forum.world.st/The-Inbox-System-jr-919-mcz-td4933985.html > > cheers -ben > > On Sun, May 21, 2017 at 5:12 PM, Thierry Goubier <[email protected]> wrote:
> Hi Ben, > > There is two Traits tests in the FileTree repository, and they pass (both > under filetree and under gitfiletree) even if I notice that you can load > the test package fine, but you get a DNU when browsing that package from > the test repository inspector. > > The tests classes are MCFileTreeLoaderTraitsTest and > MCGitFileTreeLoaderTraitsTest, and the test repository is at: > https://github.com/dalehenrich/filetree/tree/pharo5.0/tests/ > testRepositories/traits. > > Would you be interested in checking if the Squeak bug applies, adding an > additional test to confirm or infirm with a pull request? > > I guess I could if no one else picks it up, but I don't think I'm best placed to do it. I've never used FileTree nor Traits, so I'm not sure where to look for the Trait's comment (is this just a class comment on a subclass of Trait) - and given the move to Iceberg/Git, I'm not sure the FileTree/Git learning curve is worth it for me. I was hoping someone familiar with both or one of them could at least confirm the problem first. cheers -ben
