git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Patrick Steinhardt <ps@pks.im>
To: Usman Akinyemi <usmanakinyemi202@gmail.com>
Cc: git@vger.kernel.org,
	Phillip Wood Phillip Wood <phillip.wood@dunelm.org.uk>,
	Christian Couder <christian.couder@gmail.com>,
	Taylor Blau <me@ttaylorr.com>,
	gitster@pobox.com
Subject: Re: [Outreachy][proposal]: Finish adding a 'os-version' capability to Git protocol v2
Date: Mon, 28 Oct 2024 07:10:54 +0100	[thread overview]
Message-ID: <Zx8q7jnDIS3_omBm@pks.im> (raw)
In-Reply-To: <CAPSxiM-YAAMwOvH8KYO+qKahCBHgw-NDb-eHJKNCZyk8xtpeGQ@mail.gmail.com>

On Mon, Oct 28, 2024 at 01:56:36AM +0000, Usman Akinyemi wrote:
> Hello,
> 
> As I planned to submit a proposal for two projects, below is my
> proposal for "Conversion of Git Unit Tests to Clar Testing Framework".
> I would just go straight into the project proposals as the other
> necessary information is in the previous email. I would prefer to work
> on the "Finish adding a 'os-version' capability to Git protocol v2"
> project in case.

Thanks for your applications! I'll only reply to this email as I'm not
mentoring the other project.

[snip]
> ---- December 23 - February 10: Incremental Conversions & Feedback
> Integration -----
> 
> - File Renaming: Rename test files from t-<name>.c to <name>.c
> following the structure outlined by prior contributions.
> - Clar Suite Integration: Add the module name to CLAR_TEST_SUITE in
> the Makefile to include the file in the Clar testing suite.
> - Conversion of Test Assertions: Replace the test assertions with
> Clar’s built-in functions (cl_assert, cl_assert_equal_s,
> cl_assert_equal_i, etc.), enhancing readability and functionality.
> -  Patch Submission: Submit each converted test as an incremental
> patch to Git's mailing list following the Git contribution guidelines.
> This phased approach enables targeted feedback from the community.

One thing missing here is the work with upstream in case we want to
implement any functionality that doesn't yet exist in clar. You mention
it in "Additional Goals and Extensions" though, so that might be good
enough.

Other than that this looks good to me, thanks! Please don't forget to
upload your applications to Outreachy if you haven't done so already.

Patrick

  reply	other threads:[~2024-10-28  6:11 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-10-27 14:53 [Outreachy][proposal]: Finish adding a 'os-version' capability to Git protocol v2 Usman Akinyemi
2024-10-28  1:56 ` Usman Akinyemi
2024-10-28  6:10   ` Patrick Steinhardt [this message]
2024-10-28  6:20     ` Usman Akinyemi
2024-10-28  7:30       ` Patrick Steinhardt
2024-10-28  8:26 ` Christian Couder
2024-10-28  9:09   ` Usman Akinyemi
2024-10-28  9:27     ` Usman Akinyemi
2024-10-29  9:13       ` Christian Couder
2024-10-29  9:52         ` Usman Akinyemi
2024-10-29 11:50           ` Usman Akinyemi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Zx8q7jnDIS3_omBm@pks.im \
    --to=ps@pks.im \
    --cc=christian.couder@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=me@ttaylorr.com \
    --cc=phillip.wood@dunelm.org.uk \
    --cc=usmanakinyemi202@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).