rglullis@communick.news to Experienced DevsEnglish · 2 days agoWhile reviewing a PR, you find some piece of code that seems to work perfectly well, but some functions are written in a style that you don't particularly favor. What do you do?cupid.careersexternal-linkmessage-square15fedilinkarrow-up125arrow-down11
arrow-up124arrow-down1external-linkWhile reviewing a PR, you find some piece of code that seems to work perfectly well, but some functions are written in a style that you don't particularly favor. What do you do?cupid.careersrglullis@communick.news to Experienced DevsEnglish · 2 days agomessage-square15fedilink
minus-squaresubignition@fedia.iolinkfedilinkarrow-up8·1 day agoIf a style guide exists, then style is important enough that this needs to be discussed with the team if it’s an edge case that isn’t covered by the guide. Not a programmer. Just love documentation. 😇
If a style guide exists, then style is important enough that this needs to be discussed with the team if it’s an edge case that isn’t covered by the guide.
Not a programmer. Just love documentation. 😇