Comment on Manager: This task only takes 30 minutes. Why did it take you the whole day?
paul@techy.news 11 months ago
do git commit -v and then just summarize the diff you have in your editor in a human readable form.
Comment on Manager: This task only takes 30 minutes. Why did it take you the whole day?
paul@techy.news 11 months ago
do git commit -v and then just summarize the diff you have in your editor in a human readable form.
KeepFlying@lemmy.world 11 months ago
Don’t just summarize the content though, summarize the rationale or how things connect. I can read your diff myself to see what changed, I want to know the logical connections, the reason you did X and not Y, etc.
Or just say “stuff” and provide that context in the PR description separately, no need to overdo the commit log on a feature branch if you’re using squash merges from your PR.
deadbeef79000@lemmy.nz 11 months ago
1000x this.
I can read a diff.
I need to know why.
No, a code comment isn’t good enough, it’s out the f date after the next commit.