• @pscamodio@feddit.it
    link
    fedilink
    251 month ago

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

    eslint . --max-warnings 0
    
      • @pscamodio@feddit.it
        link
        fedilink
        31 month 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.

        • @jjjalljs@ttrpg.network
          link
          fedilink
          21 month ago

          Several times I’ve set the max warnings to whatever the current warning count is, and then decreased that over time.