| ▲ | russfink a day ago | |
Instead of accepting the default “commit then push,” try “draft a commit message and let me review and revise it first.” | ||
| ▲ | tepmoc 6 hours ago | parent | next [-] | |
I surprised how many people don’t even read their own PR commits and diffs, like at basic level. So many don’t even bother to tide-up fold their commits into single logical change, consider how many commit agent doing, seems logical to have folded commit history when you open PR. Default commit and PR body text is word salad didn’t seems to bothered these people. Because if it did they already had hooks and skills to make it look tidy and lack of any AI attribution inserted. I don’t even consider myself developer more like really weak devops guy, but I was already doing all of that. | ||
| ▲ | verdverm 21 hours ago | parent | prev [-] | |
one can go as far as preventing pushes altogether, or don't give them write permissions to your repos | ||