From: | Yosry Muhammad <yosrym93(at)gmail(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org>, Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>, pgadmin-hackers(at)postgresql(dot)org |
Subject: | Re: [GSoC][Patch] Automatic Mode Detection V1 |
Date: | 2019-06-25 15:19:53 |
Message-ID: | CAFSMqn9RP09oiCf1H7=8YKVOE2uszFBTPNSDJYMLyFM7qQsz8g@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Tue, Jun 25, 2019 at 1:09 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
> - What else is missing from this patch to make it applicable ? I would
>>> like to produce a release-ready patch if possible. If so, I can continue
>>> working on the project on following patches, I just want to know what is
>>> the minimum amount of work needed to make this patch release-ready
>>> (especially that changes are being made in the master branch that require
>>> me to re-edit parts of the code that I have written before to keep things
>>> in-sync).
>>>
>> @Dave Page is the right person to answer this.
>>
>
> It needs:
>
> - A code complete feature (or infrastructure/refactoring ready for a
> feature), that is acceptable to us. Seems like this is 90% there for an
> initial commit.
> - Documentation updates.
> - Tests for the new feature to ensure it works without needing manual
> testing.
> - To pass all existing tests (which may be modified if appropriate).
>
>
Could you tell me what is missing from this patch (in terms of features -
other than tests) to be acceptable? I will start working on the tests once
the patch is complete. The patch passes all the existing tests except for 3
feature tests that fail due to a TimeoutException in selenium. I do not
know what this is about I hope Aditya will help me with it.
Also, do you mean code documentation or documentation for the users? Could
you point me towards the related parts?
Thanks a lot.
--
*Yosry Muhammad Yosry*
Computer Engineering student,
The Faculty of Engineering,
Cairo University (2021).
Class representative of CMP 2021.
https://www.linkedin.com/in/yosrym93/
From | Date | Subject | |
---|---|---|---|
Next Message | Alex Kim | 2019-06-25 23:32:24 | Getting Involved |
Previous Message | Yosry Muhammad | 2019-06-25 15:16:32 | Re: [GSoC][Patch] Automatic Mode Detection V1 |