git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Re: [GSOC] Interested in the project 'Unify ref-filter formats with other pretty formats'
       [not found] <CANsrJQd1H-NXVA69HxkzH1ZODTKaLzp-P7B7x+Qpbm5wyJFeDg@mail.gmail.com>
@ 2022-03-10 23:22 ` Hariom verma
       [not found]   ` <CANsrJQdaix6=q=hXeB2kjUW+OTiXhsSCJom5ppa_SjzQaV5G8A@mail.gmail.com>
  0 siblings, 1 reply; 2+ messages in thread
From: Hariom verma @ 2022-03-10 23:22 UTC (permalink / raw)
  To: Jayati Shrivastava; +Cc: christian.couder, Git List

Hi Jayati,

Welcome to Git.

On Fri, Mar 11, 2022 at 12:52 AM Jayati Shrivastava <gaurijove@gmail.com> wrote:
>
> Hi,
>
> I am Jayati Shrivastava, a senior undergrad at Indian Institute of
> Technology Roorkee where I am majoring in Electrical Engineering with
> minors in Computer Science. I am interested in working on the project
> 'Unify ref-filter formats with other pretty formats' under GSOC'22 and
> have been exploring Hariom's and Olga's prior work on it. I have
> previously contributed to open-source communities like Mozilla
> Firefox, Facebook's Jest, phpMyAdmin and Apache Airflow. I am very
> enthusiastic about joining the Git community and would love to
> contribute to this project under your mentorship.

Great.

> If the microproject 'Use test_path_is_* functions in test scripts' is available,
> I would like to submit a patch for it. Please let me know what you think.

Sure. You can pick this up.

> Regards,
> Jayati

Thanks,
Hariom

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [GSOC] Interested in the project 'Unify ref-filter formats with other pretty formats'
       [not found]   ` <CANsrJQdaix6=q=hXeB2kjUW+OTiXhsSCJom5ppa_SjzQaV5G8A@mail.gmail.com>
@ 2022-03-11 21:48     ` Hariom verma
  0 siblings, 0 replies; 2+ messages in thread
From: Hariom verma @ 2022-03-11 21:48 UTC (permalink / raw)
  To: Jayati Shrivastava; +Cc: Christian Couder, git

On Fri, Mar 11, 2022 at 9:54 PM Jayati Shrivastava <gaurijove@gmail.com> wrote:
>
> For the microproject - "Use test_path_is_* functions in test scripts", is the intent
> to do away with all calls to run_command*  or to only remove those run_command*
> calls where spawning a child process is an overkill (similar to https://github.com/git/git/commit/ffcb4e94d3) ?
> If it's the latter then I am not sure how to identify such cases.
> Would be great if you could clarify it further.

Since `test_path_is_*` functions are more robust and debug-friendly.
And are just wrappers around `test -(e|f|d|…)`. So, I think it's okay
to use them wherever possible.

But maybe someone on the mailing list will be able to provide better
and correct explanation. cc(ing) Christian and mailing list.

> Meanwhile, I'll try to work on another microproject to get familiar
> with the patch submission process.

We generally expect one microproject per student. But contributions
are always welcome :)

Thanks,
Hariom

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2022-03-11 22:56 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <CANsrJQd1H-NXVA69HxkzH1ZODTKaLzp-P7B7x+Qpbm5wyJFeDg@mail.gmail.com>
2022-03-10 23:22 ` [GSOC] Interested in the project 'Unify ref-filter formats with other pretty formats' Hariom verma
     [not found]   ` <CANsrJQdaix6=q=hXeB2kjUW+OTiXhsSCJom5ppa_SjzQaV5G8A@mail.gmail.com>
2022-03-11 21:48     ` Hariom verma

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).