10

Never had a coding style argument because my workplace doesn't have any standards, mix of C# and VB sure why not, 2K line VB service with 6 comments total? Sure no problems there.

The only styling we have is our personal preferences, except when it's my projects, then every adheres to the styles I set because I won't merge their 800 line monstrosity of a file with 18 classes.

Comments
  • 3
    wait one file 18 classes, please tell me I'm wrong
  • 2
    @gitpush nope not wrong, this is actually something I've seen
  • 1
    @tankerkiller125 wow, what .... the .... fk..........
  • 1
    @gitpush I promise you that's not even the worse thing I've seen. A lot of the devs are former designers turned devs out of necessity during the 2008 crash and the company just never reverted them back to design and hasn't hired a new Dev in a couple of years. I'm just the IT guy but they keep trying to drag me into customer projects.
  • 0
    @tankerkiller125 You better stick to your IT job, that source code is a ticking time bomb that might blow up any second, don't be the poor soul that is next to that bomb when it blows up
  • 1
    @gitpush I'm rewriting all the internal stuff myself using good code practices, I'm staying away from the customer stuff though
  • 2
    My company uses the scalastyle checker but the way that one of my colleagues fixes the warnings is to ignore them.
  • 0
  • 1
    @magicMirror it's all good. The new comments for ignoring the warnings are easy to spot in pull requests and ask why they're there. Not so easy for him to justify though.
Add Comment