undefined | Better HN

0 pointsIzkata1y ago0 comments

> Set up your tooling to make the process smooth.

> Leverage automation. Run [..] linters, static checkers [..]

These don't make the process smooth unless you set them up to simply give a warning rather than block the build/merge. And with that they'll likely get ignored anyway.

I think linters/etc should be avoided until you already have buy-in from the team.

0 comments

azov1y ago

It depends. If your codebase is already free of lint warnings - adding a blocking check to prevent new ones is no big deal. But if your blocking check means that everyone has to drop everything and spend a week fixing code - of course this won't be smooth.

PS. Also, it’s a good idea to have manual override for whatever autoblocks you set up. Most linters already come with this feature.

ElevenLathe1y ago

IME even better than linters is something like `go fmt` (I work mainly in Python and use https://github.com/psf/black). Rather than forcing you to manually jimmy the code to satisfy the robot, just let the robot make it look the way it wants.

j / k navigate · click thread line to collapse

0 comments

azov1y ago

PS. Also, it’s a good idea to have manual override for whatever autoblocks you set up. Most linters already come with this feature.

ElevenLathe1y ago

j / k navigate · click thread line to collapse