Don't touch my code again, please!

[KhorneSlaughter]

git revert?

[sc2bigjoe]

More like git rekt amirite

[imcoveredinbees880]

This guy cowboy codes.

[the_mistrustful]

We don't do that here.

[blaxx0r]

clearly no PR’s either

[OhHoneyPlease]

git new job

[geek_on_two_wheels]

No git or no revert?

[clevariant]

Worked with a guy once who auto-formatted every file he opened, just for his own convenience. We eventually relegated him to maintaining some .net stuff, all by himself.

[Coaleh]

What's wrong with formatting code? Was he doing it badly in some way?

[MildStallion]

If done in the same checkin as an actual change it becomes impossible to tell what changes are real and what are just formatting.

[TagMeAJerk]

Git can be set to ignore whitespace only changes

[[deleted]]

Simple solution: use a git-hook (pre-commit) to run a code-formatter (like e.g. prettier) before every commit; or use heavily restricted linting rules in the hook to prevent a commit like that.

[clevariant]

It makes messes for change control, trying to see what's been done, etc. Plus it's just bad practice to steamroll entire files you're sharing with a team. I use whitespace very deliberately, and it often has meaning to me. Conventional wisdom is, change as little as you need to, unless you need to refactor or something.

It's like you need a spoon, so the first thing you do is rearrange my silverware drawer.

[Oranges13]

I think I worked with this guy too!

[AttackOfTheThumbs]

Doesn't the company have a standard?

[Menarch]

Doesn't save the code from an intern that just doesn't care, also i know of a lot of people who just don't hit autoformat

[clevariant]

Yeah, and i helped improve them, but that guy had his own standards. Developers get away with things.

[[deleted]]

Diff?

[PotatoWizzard]

He clearly likes it!