762
looks good to me (lemm.ee)
submitted 2 years ago by to c/programmer_humor@programming.dev
47 comments fedilink hide all child comments
 
you are viewing a single comment's thread
view the rest of the comments
[–] 25 points 2 years ago (1 child)

Meanwhile in another universe one of my biggest win was to introduce this line in our PR validation pipeline.

eslint . --max-warnings 0
  • permalink
  • fedilink
  • source
  • hideshow 2 child comments
    •
  • [+] 3 points 2 years ago* (last edited 3 months ago) (1 child)
    [deleted]
  • permalink
  • fedilink
  • source
  • parent
  • hideshow 2 child comments
    •
  • [–] 3 points 2 years ago (1 child)

    In a codebase with a lot of warnings is even better for me to add a disable comments for all the existing warning and then not allow any new one in.

    And then each time a part of the code needs to be touched the existing warning there should be solved too.

  • permalink
  • fedilink
  • source
  • parent
  • hideshow 2 child comments
    •