Thread (27 messages) 27 messages, 6 authors, 2022-02-24

Re: [PATCH 0/2] microproject: avoid using pipes in test

From: Christian Couder <hidden>
Date: 2022-02-24 09:29:42

On Thu, Feb 24, 2022 at 6:22 AM Shubham Mishra [off-list ref] wrote:
quoted
quoted
This is my first contribution that's why I am keeping diff short with an intention to understand the process instead of making impactful change in first attempt. Later, I will send more patches to fix the issue for other files.
Please read the "Only ONE quality focused microproject per student"
section of the above mentioned page.
here I mean in next patches, I will be fixing the same "Git on LHS of
pipe" for other tests. I think that will be considered as the same
microproject and I am still eligible to fix the same thing for other
files?
In the "Only ONE quality focused microproject per student" section, there is:

=> This means that for a microproject that consist in refactoring or
rewriting a small amount of code, your patch should change only ONE
file, or perhaps 2 files if they are closely related, like “foo.c” and
“foo.h”.

So, no, we really prefer you to focus on 1 file (or 2 files when they
are strongly related) and then move on to your application or regular
patches, reviews, discussions, etc.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help