{"thread":{"id":"61175","subject":"GSoC 2024","startedAt":"2024-03-22T06:58:08Z","lastAt":"2024-03-26T11:22:35Z","messageCount":4,"participants":["Aishwarya Narayanan","Karthik Nayak","Patrick Steinhardt"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"491220","messageId":"CAHCXyj3U69qyhYewOLY9hN2rvi_5ZuSxQEBJbDxrKefm9MzVWg@mail.gmail.com","threadId":"61175","inReplyTo":null,"subject":"GSoC 2024","fromName":"Aishwarya Narayanan","fromEmail":"aishnana.03@gmail.com","sentAt":"2024-03-22T06:57:59Z","receivedAt":"2024-03-22T06:58:08Z","isPatch":false,"sender":{"key":"aishnana.03@gmail.com","avatar":null},"body":"Dear Git Organization,\n\nI hope this email finds you well. I am reaching out regarding a\nRequest for Assistance with Microproject aimed at improving the\nintegration test scripts in the Git project by avoiding the\nsuppression of Git exit codes. As a beginner, I would greatly\nappreciate your guidance on how to approach this task effectively.\n\nHere's a breakdown of the microproject and how I plan to tackle it as\na beginner:\n\nUnderstanding the Problem:\n\nThe Git project utilizes Shell scripts for integration testing to\nensure software stability.\nIt's crucial to avoid suppressing the exit codes of Git commands in\nthese scripts, as it may lead to errors going unnoticed.\n\nIdentifying Problematic Patterns:\n\nI will examine the integration test scripts located in the t directory\nof the Git project.\nSpecifically, I will search for instances where Git commands are piped\ninto other commands, or where command substitution is used with Git\ncommands.\n\nLearning from Examples:\n\nI will refer to the provided examples of bad and good practices to\nunderstand what to look for and how to correct it.\nUnderstanding why certain practices are bad and others are good will\nhelp me grasp the underlying concepts better.\n\nApplying Correct Practices:\n\nWhen I identify instances of suppressed exit codes, I will modify the\nscripts to adhere to the good practices outlined.\nThis may involve replacing problematic commands with alternatives that\nproperly handle Git exit codes.\n\nValidation and Documentation:\n\nAfter making modifications, I will ensure that the scripts still\nfunction correctly by running the affected integration tests.\nI will document the changes made and provide explanations for why\ncertain commands were modified. This documentation will aid in\nunderstanding the improvements made to the test suite.\n\nSeeking Feedback:\n\nOnce the changes are implemented, I will seek feedback from mentors or\ncollaborators to ensure that the modifications are in line with\nproject standards.\n\nI would greatly appreciate any insights or advice you can offer to\nhelp me navigate through this microproject effectively. Additionally,\nif you could provide guidance on where to find another test script\nsimilar to the one referenced in commit c6f44e1da5, it would be\nimmensely helpful.\n\nThank you very much for your time and assistance. I look forward to\nyour guidance.\n\nBest regards,\nAishwarya Narayanan\n"},{"id":"491482","messageId":"CAOLa=ZRqkKhqUPjrYQWmLnLAf6Cmt17ZSMJja0rYLNWifUG+og@mail.gmail.com","threadId":"61175","inReplyTo":"CAHCXyj3U69qyhYewOLY9hN2rvi_5ZuSxQEBJbDxrKefm9MzVWg@mail.gmail.com","subject":"Re: GSoC 2024","fromName":"Karthik Nayak","fromEmail":"karthik.188@gmail.com","sentAt":"2024-03-25T17:45:38Z","receivedAt":"2024-03-25T17:45:40Z","isPatch":false,"sender":{"key":"karthik.188@gmail.com","avatar":"https://avatars.githubusercontent.com/u/1786334?v=4"},"body":"Aishwarya Narayanan <aishnana.03@gmail.com> writes:\n\nHello,\n\n> Dear Git Organization,\n>\n> I hope this email finds you well. I am reaching out regarding a\n> Request for Assistance with Microproject aimed at improving the\n> integration test scripts in the Git project by avoiding the\n> suppression of Git exit codes. As a beginner, I would greatly\n> appreciate your guidance on how to approach this task effectively.\n>\n\nThanks for your interest and reaching out.\n\n> Here's a breakdown of the microproject and how I plan to tackle it as\n> a beginner:\n>\n> Understanding the Problem:\n>\n> The Git project utilizes Shell scripts for integration testing to\n> ensure software stability.\n> It's crucial to avoid suppressing the exit codes of Git commands in\n> these scripts, as it may lead to errors going unnoticed.\n>\n\nCorrect.\n\n> Identifying Problematic Patterns:\n>\n> I will examine the integration test scripts located in the t directory\n> of the Git project.\n> Specifically, I will search for instances where Git commands are piped\n> into other commands, or where command substitution is used with Git\n> commands.\n>\n\nThe goal of the microproject is to help you get started with\ncontributing to the Git codebase. As such, it is recommended that you\nfocus on finding and fixing a single instance only.\n\n> Learning from Examples:\n>\n> I will refer to the provided examples of bad and good practices to\n> understand what to look for and how to correct it.\n> Understanding why certain practices are bad and others are good will\n> help me grasp the underlying concepts better.\n>\n> Applying Correct Practices:\n>\n> When I identify instances of suppressed exit codes, I will modify the\n> scripts to adhere to the good practices outlined.\n> This may involve replacing problematic commands with alternatives that\n> properly handle Git exit codes.\n>\n\nI would also recommend reading the following documents:\n- Documentation/SubmittingPatches\n- Documentation/MyFirstContribution.txt\n\n> Validation and Documentation:\n>\n> After making modifications, I will ensure that the scripts still\n> function correctly by running the affected integration tests.\n> I will document the changes made and provide explanations for why\n> certain commands were modified. This documentation will aid in\n> understanding the improvements made to the test suite.\n>\n> Seeking Feedback:\n>\n> Once the changes are implemented, I will seek feedback from mentors or\n> collaborators to ensure that the modifications are in line with\n> project standards.\n>\n> I would greatly appreciate any insights or advice you can offer to\n> help me navigate through this microproject effectively. Additionally,\n> if you could provide guidance on where to find another test script\n> similar to the one referenced in commit c6f44e1da5, it would be\n> immensely helpful.\n>\n\nWell, you could use a regex search to find some of them. This one is for\nfinding tests with pipes used, but do note that you could get false\npositives here too:\n\n    grep -Er \"git .* \\|\" t/\n\n>\n> Thank you very much for your time and assistance. I look forward to\n> your guidance.\n>\n> Best regards,\n> Aishwarya Narayanan\n\nWish you luck!\n\n- Karthik\n"},{"id":"491562","messageId":"CAHCXyj2a5QFO+fDekR0sQuyFLocucMQeuABxG=6g1X92_cEkpA@mail.gmail.com","threadId":"61175","inReplyTo":"CAOLa=ZRqkKhqUPjrYQWmLnLAf6Cmt17ZSMJja0rYLNWifUG+og@mail.gmail.com","subject":"Re: GSoC 2024","fromName":"Aishwarya Narayanan","fromEmail":"aishnana.03@gmail.com","sentAt":"2024-03-26T09:03:59Z","receivedAt":"2024-03-26T09:04:04Z","isPatch":false,"sender":{"key":"aishnana.03@gmail.com","avatar":null},"body":"Dear Git Project Maintainers,\n\nAssistance Needed: Locating Test Script for Addressing Git's Exit Code\nSuppression. I hope this email finds you well. My name is Aishwarya,\nand I am reaching out to seek assistance with a microproject I am\nworking on within the Git project.\n\nAs part of my efforts to contribute to the Git project, I am currently\nfocusing on addressing the suppression of Git's exit code in test\nscripts, as outlined in the project's microproject ideas.\nSpecifically, I aim to identify a test script within the t directory\nwhere Git's exit code may be inadvertently suppressed.\n\nWhile I have attempted to locate such a test script independently, I\nhave encountered challenges in identifying a suitable candidate.\nTherefore, I am reaching out to request guidance from the project\nmaintainers in locating a test script where this issue needs to be\naddressed.\nI tried commands that included:\ngrep -Er \"git .* \\|\" t/, grep -R \"git .* |\" t/, findstr /s /i /p \"git .* |\" *.sh\n\nCould you please provide any insights or guidance on identifying a\ntest script within the t directory that exhibits the suppression of\nGit's exit code? Any pointers or suggestions would be greatly\nappreciated.\n\nAdditionally, if there are specific criteria or patterns that I should\nlook for when identifying such test scripts, please let me know, and I\nwill ensure to align my search accordingly.\n\nThank you very much for your time and assistance. I am eager to\ncontribute to the Git project and look forward to your guidance in\nthis matter.\n\n\nBest regards,\nAishwarya Narayanan\n\n\nOn Mon, 25 Mar 2024 at 23:15, Karthik Nayak <karthik.188@gmail.com> wrote:\n>\n> Aishwarya Narayanan <aishnana.03@gmail.com> writes:\n>\n> Hello,\n>\n> > Dear Git Organization,\n> >\n> > I hope this email finds you well. I am reaching out regarding a\n> > Request for Assistance with Microproject aimed at improving the\n> > integration test scripts in the Git project by avoiding the\n> > suppression of Git exit codes. As a beginner, I would greatly\n> > appreciate your guidance on how to approach this task effectively.\n> >\n>\n> Thanks for your interest and reaching out.\n>\n> > Here's a breakdown of the microproject and how I plan to tackle it as\n> > a beginner:\n> >\n> > Understanding the Problem:\n> >\n> > The Git project utilizes Shell scripts for integration testing to\n> > ensure software stability.\n> > It's crucial to avoid suppressing the exit codes of Git commands in\n> > these scripts, as it may lead to errors going unnoticed.\n> >\n>\n> Correct.\n>\n> > Identifying Problematic Patterns:\n> >\n> > I will examine the integration test scripts located in the t directory\n> > of the Git project.\n> > Specifically, I will search for instances where Git commands are piped\n> > into other commands, or where command substitution is used with Git\n> > commands.\n> >\n>\n> The goal of the microproject is to help you get started with\n> contributing to the Git codebase. As such, it is recommended that you\n> focus on finding and fixing a single instance only.\n>\n> > Learning from Examples:\n> >\n> > I will refer to the provided examples of bad and good practices to\n> > understand what to look for and how to correct it.\n> > Understanding why certain practices are bad and others are good will\n> > help me grasp the underlying concepts better.\n> >\n> > Applying Correct Practices:\n> >\n> > When I identify instances of suppressed exit codes, I will modify the\n> > scripts to adhere to the good practices outlined.\n> > This may involve replacing problematic commands with alternatives that\n> > properly handle Git exit codes.\n> >\n>\n> I would also recommend reading the following documents:\n> - Documentation/SubmittingPatches\n> - Documentation/MyFirstContribution.txt\n>\n> > Validation and Documentation:\n> >\n> > After making modifications, I will ensure that the scripts still\n> > function correctly by running the affected integration tests.\n> > I will document the changes made and provide explanations for why\n> > certain commands were modified. This documentation will aid in\n> > understanding the improvements made to the test suite.\n> >\n> > Seeking Feedback:\n> >\n> > Once the changes are implemented, I will seek feedback from mentors or\n> > collaborators to ensure that the modifications are in line with\n> > project standards.\n> >\n> > I would greatly appreciate any insights or advice you can offer to\n> > help me navigate through this microproject effectively. Additionally,\n> > if you could provide guidance on where to find another test script\n> > similar to the one referenced in commit c6f44e1da5, it would be\n> > immensely helpful.\n> >\n>\n> Well, you could use a regex search to find some of them. This one is for\n> finding tests with pipes used, but do note that you could get false\n> positives here too:\n>\n>     grep -Er \"git .* \\|\" t/\n>\n> >\n> > Thank you very much for your time and assistance. I look forward to\n> > your guidance.\n> >\n> > Best regards,\n> > Aishwarya Narayanan\n>\n> Wish you luck!\n>\n> - Karthik\n"},{"id":"491566","messageId":"ZgKv9sZ6shfqym2P@tanuki","threadId":"61175","inReplyTo":"CAHCXyj2a5QFO+fDekR0sQuyFLocucMQeuABxG=6g1X92_cEkpA@mail.gmail.com","subject":"Re: GSoC 2024","fromName":"Patrick Steinhardt","fromEmail":"ps@pks.im","sentAt":"2024-03-26T11:22:30Z","receivedAt":"2024-03-26T11:22:35Z","isPatch":false,"sender":{"key":"ps@pks.im","avatar":"https://avatars.githubusercontent.com/u/4056630?v=4"},"body":"On Tue, Mar 26, 2024 at 02:33:59PM +0530, Aishwarya Narayanan wrote:\n> Dear Git Project Maintainers,\n> \n> Assistance Needed: Locating Test Script for Addressing Git's Exit Code\n> Suppression. I hope this email finds you well. My name is Aishwarya,\n> and I am reaching out to seek assistance with a microproject I am\n> working on within the Git project.\n\nPlease not that top-posting is discouraged on the Git mailing list [1].\n\n> As part of my efforts to contribute to the Git project, I am currently\n> focusing on addressing the suppression of Git's exit code in test\n> scripts, as outlined in the project's microproject ideas.\n> Specifically, I aim to identify a test script within the t directory\n> where Git's exit code may be inadvertently suppressed.\n> \n> While I have attempted to locate such a test script independently, I\n> have encountered challenges in identifying a suitable candidate.\n> Therefore, I am reaching out to request guidance from the project\n> maintainers in locating a test script where this issue needs to be\n> addressed.\n> I tried commands that included:\n> grep -Er \"git .* \\|\" t/, grep -R \"git .* |\" t/, findstr /s /i /p \"git .* |\" *.sh\n> \n> Could you please provide any insights or guidance on identifying a\n> test script within the t directory that exhibits the suppression of\n> Git's exit code? Any pointers or suggestions would be greatly\n> appreciated.\n> \n> Additionally, if there are specific criteria or patterns that I should\n> look for when identifying such test scripts, please let me know, and I\n> will ensure to align my search accordingly.\n\nThe command proposed by Karthik (which is also part of your pipeline\nabove) does result in several matches in \"t/\", some of which are actual\ngood candidates for a micro project. Did you scan through these matches?\nThe microprojects page at [2] should give you a good idea for what to be\non the lookout for as it explains the issue as well as highlighting some\npatterns to watch out for.\n\nPatrick\n\n[1]: https://www.mediawiki.org/wiki/Mailing_list_etiquette\n[2]: https://git.github.io/SoC-2024-Microprojects/\n"}]}