This is my second post for the series hacktoberfest. We will also be talking about my second contribution in an open source project for hacktoberfest.
This time I worked on a something more complex as opposed to my last contribution which was just documentation. For this issue, I was supposed to build a CI/CD pipeline for the repository from scratch. The purpose of this would be to lint check a project before a PR is made.
I had to do a little bit of research for this because this was my first time setting up a CI/CD pipeline on GitHub actions. After a bit of a research I was able to make the new lint.yml file and raise a PR. This was actually very fun, because I was able to find some amazing documentation for this which made my task so much more easier than it would normally be.
I got a review that I was using old versions of commands. Turns out the docs I was referring to were old. Also I did not add caching for npm packages. Add these was not hard, I just had to make changes in three more lines. But this made me realize that us a contributors are not only supposed get the issue done, but also make sure that we are using the best practises so that the code is more maintainable and sustainable.
This is my main takeaway from my contribution in this project, which is to make sure we are not only completing the issue but doing it with best practises in mind. In order to achieve this, do some extra research if you have to,
Lastly below is the code I added for quick reference,
Top comments (0)