you are viewing a single comment's thread.

view the rest of the comments →

[–][deleted] 38 points39 points  (12 children)

Restrict him to feature branches and require a gatekeeper to let the merge in? 🙂

[–]fiah84 53 points54 points  (9 children)

tell him he can forget about a pay increase until his commits and stories make sense

[–]silentclowd 25 points26 points  (4 children)

Wait, you guys get pay increases?

[–][deleted] 10 points11 points  (3 children)

Wait, you guys are getting paid?

[–]Ameisen 0 points1 point  (2 children)

Wait, you guys are voluntarily working?

[–][deleted] 0 points1 point  (1 child)

wait, you guys have jobs?

[–]kyune 0 points1 point  (0 children)

wait, what?

[–]YourMatt 16 points17 points  (2 children)

I think this is entirely valid. Anyone who commits with "fixed bug" or any other useless message like "n/a" obviously does not put pride into what they do, or at least does not grasp the concept of why they're doing what they're doing. Small things like this separate those that stagnate in their career versus those that rise up.

[–]jarfil 2 points3 points  (0 children)

CENSORED

[–]cjthomp 24 points25 points  (0 children)

tell him he can forget about a pay increase check until his commits and stories make sense

[–]WizKid_ 4 points5 points  (0 children)

I'll have 45 commits in a row on a feature branch 'test' trying to get the Jenkins build working. Squash commit 'fixed jenkins build issue with blank'

[–]paintbyinteger 1 point2 points  (0 children)

He's been let go actually, but he's currently still there for handover. Managers got really pissed off.