First Open Source PR Merged

Krinskumar Vaghasia - Sep 19 - - Dev Community

In lab 2, the students were supposed to make contributions in open source projects made by other students. I decided to work onCode Formatter Advisor, which uses Groq to analyze code and provide formatting improvements.

I also tested this application for my previous lab so I had a fair bit on understanding to how the it works. In the lab we were supposed to add a new command line flag which shall display the token information. All of this while following the guidelines for contribution.

I had done this before, so it was very easy for make a new issue->fork->checkout to a new branch->commit->push->pr->merge. For the implementation part, I worked with grok as well, so I knew where to look for token info, the hard part for me was to get used to the tool in python. I made my tool in javascript. This tool used the argparse python library which was not something I was aware of. It took some time for me to get used to it let along add a new flag. However, once I got used to library it was pretty smooth. In terms of changes I added a new flag for the function to parse and added a new logic for what happens when the flag is passed. Getting the PR approved was fairly simple, cuz all of new changes were working correctly for owner.

Adding images in the PR is a good way to make sure everyone is on the same page, therefore I added an image of the final state of the command line with the flag turned on!!.

Image description

If I were to do the whole process again, I will make sure I understand how every library used works because I dove right in for this project and wasted a lot of time.

. . . . . . . . . . . . . . . . . . . . .
Terabox Video Player