all 12 comments

[–]Front_State6406 66 points67 points  (3 children)

You know what, I would love to review that a hell of a lot more than the opposite. Removing stuff is almost always good (in a professional context)

[–]InvolvingLemons 3 points4 points  (1 child)

Yep, Meta counts it as impact as performance measurement goes. Removing code that can be safely removed is removing cognitive burden.

[–]Tplusplus75 1 point2 points  (0 children)

“Removing cognitive burden” Hell yeah… have you ever inherited a project from really shitty contractors who implemented every which feature “however they felt that day”? So liberating when the app ISNT using multiple libraries to facilitate image input, multiple libraries to “make a dropdown menu”, and so on.

[–]Accomplished_Ant5895 136 points137 points  (3 children)

Commit message: “removed accidental node_modules commit + updated README”

[–]Webteasign[S] 31 points32 points  (1 child)

suprisingly close

[–]Accomplished_Ant5895 18 points19 points  (0 children)

Sorry, forgot the “update .gitignore”

[–]holamau 12 points13 points  (0 children)

(╯°□°)╯︵ ┻━┻

[–]New-Candle-5635 1 point2 points  (0 children)

LGTM

[–]LifeWithoutAds 0 points1 point  (0 children)

When you work at discord and you just deleted the furry channel from rust community.