DEV Community 👩‍💻👨‍💻

sfrunza13
sfrunza13

Posted on • Updated on

Reviewing PRs

Reviewing Pull Requests

Part of the 0.3 release was reviewing pull requests for the select Seneca applications that we are all working on together.

The pull request that I reviewed had to do with adding a prettier configuration.

The pull request included the .vscode files and an empty .prettierrc file. I wrote a comment explaining the difference between changing the settings within the editor and changing the settings through a configuration file for every contributor to be able to use in the future.

Image description

P24

Also I recently reviewed a second PR, just correcting a very small grammar mistake to test out git hub's comment functionality on PRs:

Image description

P13

What I learned

That occasionally I might have something to offer in terms of commentary/guidance and to constantly be checking the PR tab because the conversations only last so long as those PRs are open, if you wait around you can't partake.

Top comments (0)

Create an Account!
Now it's your turn!
 
🗒 Share a tutorial
🤔 Reflect on your coding journey
❓ Ask a question

Create an account to join hundreds of thousands of DEV members on their journey.