rglullis@communick.news to Experienced DevsEnglish · 3 months 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-square17fedilinkarrow-up126arrow-down11
arrow-up125arrow-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 · 3 months agomessage-square17fedilink
minus-squaresubignition@fedia.iolinkfedilinkarrow-up8·3 months 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. 😇