| ▲ | Invictus0 2 days ago | |||||||
normal guy that wants to provide for his family: bro autistic guy that rejects your PRs because you used tabs instead of spaces: pro | ||||||||
| ▲ | lukevp 2 days ago | parent | next [-] | |||||||
If you don’t have style rules baked into the PR process, you should. Tabs vs. spaces is a team decision and needs to be matched by everyone to reduce code churn. It’s correct to reject a PR if it violates the team style guide (although it should be automated and have tooling locally to enforce it as well.) | ||||||||
| ||||||||
| ▲ | 2 days ago | parent | prev | next [-] | |||||||
| [deleted] | ||||||||
| ▲ | hectdev 2 days ago | parent | prev | next [-] | |||||||
I worked for twitter and there was a guy like this. I made a big stink that "yo, everything this guy says just needs to be in a linter" and they actually implemented it. | ||||||||
| ▲ | rexpop 2 days ago | parent | prev [-] | |||||||
On the contrary, the pro is likely to prefer tabs—you can adjust the display-width for better ergonomics. | ||||||||