> On 15 Dec 2017, at 17:37, Alistair Grant <akgrant0...@gmail.com> wrote:
> 
> Hi Esteban,
> 
> I had no problems following the process (Ubuntu 16.04,
> Pharo7.0-32bit-e175bc2.image, fogbugz 20872). :-)
> 
> I guess that you have already thought of this, but...  Is there any
> reason why we can't just put up a dialog asking for the user's github
> credentials and fogbugz issue number and then automatically clone the
> repository, configure the upstream remote and create the issue branch.
> That would remove most of the remaining manual steps.
> 
> I realise that it only works for option 1, although where people
> configure a common pharo-local, it could check for a pre-existing
> clone and use that one.

"I realise” means you tried and it didn’t work?
because in my tests it worked as good as the first one (I tested on windows), 
but that may need to be “re-validated” :) 

Esteban

> 
> Cheers,
> Alistair
> 
> On 14 December 2017 at 13:19, Esteban Lorenzano <esteba...@gmail.com> wrote:
>> Hi!
>> 
>> I’m working on simplifying the contribution process, after collecting
>> opinions/experiences last couple of months.
>> As you know, Pharo contribution process is still WIP and we aim to have it
>> as smooth as possible for Pharo 7.0 release. Now, after observe the idea of
>> the “system repositories” was a bad idea because it introduced extra and non
>> standard “path” to contribution, I managed to remove that to reestablish
>> “the regular way”: you will now need to add pharo repository just as any
>> other repository you add, by cloning or adding local repository.
>> 
>> I took Guille’s doc and moved it to pharo project (it does not has sense to
>> have it living in a contributor’s repository when is so important). You can
>> find it here:
>> 
>> https://github.com/pharo-project/pharo/wiki/Contribute-a-fix-to-Pharo
>> 
>> This document is also updated to reveal this new process, please read it.
>> 
>> How to update your startup scripts?
>> Some people has added startup scripts to easy the first part of
>> contribution. Instead enabling system repositories, etc. you now need to
>> replace that with this:
>> 
>> (IceRepositoryCreator new
>> location: '/path/to/pharo-project/pharo' asFileReference;
>> subdirectory: 'src';
>> createRepository)
>> register
>> 
>> PLEASE, PLEASE, PLEASE… take a moment to read and try the document. Is very
>> important that document reflects new process and works reliable in different
>> scenarios (I validated it on macOS and Windows, and assumed it worked fine
>> on linux but you know… bad assumptions is the base of failure ;) )
>> 
>> I’m eager to hear your feedback and continue enhancing the process.
>> 
>> (yes, Stef, I know UI is still cumbersome… I’m working on that :) )
>> 
>> cheers!
>> Esteban
> 


Reply via email to