• pscamodio@feddit.it
    link
    fedilink
    arrow-up
    19
    ·
    1 day ago

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

    eslint . --max-warnings 0
    
    • skuzz@discuss.tchncs.de
      link
      fedilink
      arrow-up
      1
      ·
      3 hours ago

      Works so well, and soothes the warning annoyance brain, and keeps warnings from eventually becoming errors.

      • pscamodio@feddit.it
        link
        fedilink
        arrow-up
        1
        ·
        2 hours ago

        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.