pull down to refresh

244 sats \ 1 reply \ @SimpleStacker 8h \ parent \ on: How are you balancing newcomer training with drive-by LLM solution filtering? AI
it was surprising to me to see how many people submit PRs but say they haven't tested it. I try to test everything (as much as I can), even single line code changes, and wouldn't imagine submitting something for review without having done so.
write
preview
144 sats \ 0 replies \ @optimism 7h
Took me years to get rid of "LGTM" culture in reviews, which ultimately heightened review quality. Now the pressure is on the submitter because no one will want a fight when their review on an untested PR breaks shit
It sucks that FOSS development brings much stress and isn't always newcomer friendly but if you have a massive installed base you have to deliver quality work and avoidable debt on your main branch is going to hurt.
reply