Profile picture
Andrew Clark @acdlite
, 5 tweets, 2 min read Read on Twitter
When reviewing a PR, it’s essential that you leave a comment. Any comment. Even the PR looks great and you have no substantial feedback, find something trivial to nitpick or question. This communicates intelligence and mastery, and is widely appreciated by your colleagues.
Automated code quality tools like @PrettierCode and @geteslint can sometimes make this difficult, but with a little creativity, no PR is immune to your scrutiny.
Above all else, it’s so important to *have an opinion* about everything. Even the smallest things. Don’t let lack of knowledge or context or experience keep you from sharing your wisdom. And when you’re done sharing your wisdom with your teammates, tweet.
Indeed! Pedantry is the truest marker of genius. cc: @neiltyson
Nit: “dysfunctional” is the more common spelling
Missing some Tweet in this thread?
You can try to force a refresh.

Like this thread? Get email updates or save it to PDF!

Subscribe to Andrew Clark
Profile picture

Get real-time email alerts when new unrolls are available from this author!

This content may be removed anytime!

Twitter may remove this content at anytime, convert it as a PDF, save and print for later use!

Try unrolling a thread yourself!

how to unroll video

1) Follow Thread Reader App on Twitter so you can easily mention us!

2) Go to a Twitter thread (series of Tweets by the same owner) and mention us with a keyword "unroll" @threadreaderapp unroll

You can practice here first or read more on our help page!

Did Thread Reader help you today?

Support us! We are indie developers!


This site is made by just three indie developers on a laptop doing marketing, support and development! Read more about the story.

Become a Premium Member and get exclusive features!

Premium member ($3.00/month or $30.00/year)

Too expensive? Make a small donation by buying us coffee ($5) or help with server cost ($10)

Donate via Paypal Become our Patreon

Thank you for your support!