<div dir="ltr"><div>Hello mentors,</div><div><br></div><div>I got some feedback and I have improved my code based on it - <a href="https://github.com/Idadelveloper/rtems/blob/master/hooks/pre-commit">https://github.com/Idadelveloper/rtems/blob/master/hooks/pre-commit</a></div><div>Will love to get some more feedback and ways to make it look better.</div><div>And is it okay I start working on its documentation though it hasn't been approved yet?</div><div><br></div><div>Cheers,</div><div>Ida<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jul 1, 2021 at 6:42 PM Ida Delphine <<a href="mailto:idadelm@gmail.com">idadelm@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto">Thank you. I will make changes accordingly.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 1 Jul 2021, 5:31 pm Gedare Bloom, <<a href="mailto:gedare@rtems.org" target="_blank">gedare@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Ida,<br>
<br>
On Tue, Jun 29, 2021 at 1:11 PM Ida Delphine <<a href="mailto:idadelm@gmail.com" rel="noreferrer" target="_blank">idadelm@gmail.com</a>> wrote:<br>
><br>
> Hello mentors,<br>
><br>
> Here is the code for my pre-commit hook script. How it works is by default, upon commiting it outputs a warning stating the number of style issues in case there are mismatches.<br>
> The user can trigger the strict mode which gives a more detailed output of the style issues by running adding the mode to the config file ( git config mode "strict") - will document this.<br>
><br>
> <a href="https://github.com/Idadelveloper/rtems/blob/master/hooks/pre-commit" rel="noreferrer noreferrer" target="_blank">https://github.com/Idadelveloper/rtems/blob/master/hooks/pre-commit</a><br>
><br>
I made comments on your commit that added this:<br>
<a href="https://github.com/Idadelveloper/rtems/commit/6bfc4802d17b3aab260190d53467b750848f0002" rel="noreferrer noreferrer" target="_blank">https://github.com/Idadelveloper/rtems/commit/6bfc4802d17b3aab260190d53467b750848f0002</a><br>
<br>
> I had already sent some screenshots here on how the outcome looks like. Will love to improve my code based on your feedback and get more suggestions.<br>
><br>
><br>
> On Mon, Jun 21, 2021 at 7:05 PM Gedare Bloom <<a href="mailto:gedare@rtems.org" rel="noreferrer" target="_blank">gedare@rtems.org</a>> wrote:<br>
>><br>
>><br>
>><br>
>> On Sun, Jun 20, 2021 at 1:13 AM Ida Delphine <<a href="mailto:idadelm@gmail.com" rel="noreferrer" target="_blank">idadelm@gmail.com</a>> wrote:<br>
>>><br>
>>> Hello everyone,<br>
>>> I updated the hooks script. About the modes, we have the default, "strict" and "nonstrict" (couldn't think of better names). With the default mode, it prints a warning specifying the number of style issues if any and aborts the commit. With the strict mode, it goes into more detail showing both the formatted and unformatted patch, the number of style issues, and aborts the commit. In the non-strict mode, it simply displays the warning with the style issues and the commit happens.<br>
>>><br>
>>> The default mode basically happens when you run<br>
>>>><br>
>>>> git commit -m "Commit message"<br>
>>><br>
>>> The best method I could find to pass arguments to a script was via environment variables. So the nonstrict mode applies when you run<br>
>>>><br>
>>>> STYLEMODE=nonstrict git commit -m "Commit message"<br>
>>><br>
>>> The strict mode applies when you run<br>
>>>><br>
>>>> STYLEMODE=strict git commit -m "Commit message"<br>
>>><br>
>>><br>
>> What are the possible options to pass arguments? (Maybe, a blog post :)) Reading from a git-config file would be better than environment variables.<br>
>><br>
>> It might be better to share screenshots by a link (e.g., a blog post :)) to avoid hitting the mailing list attachment limits.<br>
>><br>
>> Gedare<br>
</blockquote></div>
</blockquote></div>