search

Leafwing-Studios / leafwing-website

The site for https://leafwing-studios.com
3 stars 2 forks source link

Add Tyranny of Nits blog post #239

Closed alice-i-cecile closed 9 months ago

alice-i-cecile commented 9 months ago

Checklist

Spot Checks

Editing passes

Social

netlify[bot] commented 9 months ago

Deploy Preview for leafwing-studios ready!

Name Link
Latest commit 091eeaef9e87f4315707ce6967702b64a8fafb6b
Latest deploy log https://app.netlify.com/sites/leafwing-studios/deploys/651624c46a6a5b00085622c3
Deploy Preview https://deploy-preview-239--leafwing-studios.netlify.app/blog
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

sixfold-origami commented 9 months ago

Oh, another thing that might be good to add is a rule of thumb I use when reviewing: If I give a piece of feedback (especially something in the polish problems or compromise conflicts areas), I'll try to write it in such a way that clearly opens room for a counterargument. If the writer provides an argument for their decision that's even somewhat reasonable, I just drop it. Usually, as a reviewer, I'm mostly wanting to check that they actually thought about the problem, and to suggest a better way if they haven't. If they have, I'm usually happy to defer to their judgement, since they actually wrote it.

alice-i-cecile commented 9 months ago

Thanks a ton for the praise :D There's been a couple typo fix PRs merged; could you check against main? If you want to be particularly nice just make a PR of your own and I'll merge it asap!