Hi Esteban, On 16 December 2017 at 09:05, Esteban Lorenzano <estebanlm@gmail.com> wrote:
On 15 Dec 2017, at 17:37, Alistair Grant <akgrant0710@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
The contribution process works fine (even on linux :-)). The "I realise" paragraph is a comment on my suggestion to try and reduce the number of manual steps required (and is actually wrong). Just to rephrase (and extend) the suggestion, I think we could create a single dialog that currently covers the following steps (from your instructions): 1. Clone a fresh repository, or point to an existing repository. 2. Tell Iceberg about pharo-project 3. Create a new branch from the fogbugz issue Cheers, Alistair
Cheers, Alistair
On 14 December 2017 at 13:19, Esteban Lorenzano <estebanlm@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