Re: [PATCH] Add ideas for GSoC 2024

From: Kaartic Sivaraam <kaartic.sivaraam@gmail.com>
Date: Feb 5, 2024, 18:55 UTC
Message-ID: <e1c04f67-5981-4393-8a8e-a28cc53858eb@gmail.com>
In-Reply-To: <CAP8UFD3F95TzytdpKO=LLf6Y_ejxwh9QtgAxRNKgMXW-72hjgQ@mail.gmail.com>

Hi Patrick, Christian and all,

On 05/02/24 22:13, Christian Couder wrote:

> 
> Thanks a lot for these ideas! I have applied your patch and pushed it.
>

Yeah. Thanks for sharing these great ideas! I've submitted the application using the new ideas page now as mentioned in the parent thread.

Show 19 quoted lines

>> +### Convert reftable unit tests to use the unit testing framework
>> +
>> +The "reftable" unit tests in "t0032-reftable-unittest.sh"
>> +predate the unit testing framework that was recently
>> +introduced into Git. These tests should be converted to use
>> +the new framework.
>> +
>> +See:
>> +
>> +  - this discussion <https://lore.kernel.org/git/cover.1692297001.git.steadmon@google.com/>
>> +
>> +Expected Project Size: 175 hours or 350 hours
>> +
>> +Difficulty: Low
> 
> "Difficulty: Low" might not be very accurate from the point of view of
> contributors. I think it's always quite difficult to contribute
> something significant to Git, and sometimes more than we expected.
>

Makes sense. Also, I'm kind of cat-one-the-wall about whether it makes sense to have two projects about the unit test migration effort itself. If we're clear that both of them would not overlap, it should be fine. Otherwise, it would be better to merge them as Patrick suggests.

That said, how helpful would it be to link the following doc in the unit testing related ideas?

https://github.com/git/git/blob/master/Documentation/technical/unit-tests.txt

>> +### Implement consistency checks for refs
>> +

 >>
 >> [ ... snip ... ]
 >>

>> +
>> +  - https://lore.kernel.org/git/6cfee0e4-3285-4f18-91ff-d097da9de737@rd10.de/
>> +  - https://lore.kernel.org/git/cover.1706601199.git.ps@pks.im/
>> +

 >> [ .... snip ... ]

>> +
>> +### Implement support for reftables in "dumb" HTTP transport

Would it worth linking the reftable technical doc for the above ideas?

https://git-scm.com/docs/reftable

I could see it goes into a lot of detail. I'm just wondering if link to it would help someone who's looking to learn about reftable.

-- 
Sivaraam