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
 
A stackoverflow answer with -5 points, it says "Adding this to main.js, worked for me." with the code being "Vue.config.silent = true;" A reply says "You're just hiding the warnings here, not actually solving the problem.-"
you are viewing a single comment's thread
view the rest of the comments
[–] 25 points 2 years ago (3 children)

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 3 child comments
    •
  • [+] 3 points 2 years ago* (last edited 4 months ago) (2 children)
    [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 1 child comment
    •