I'm new to Turborepo and I'm trying to understand some best practices. I want my builds to fail on linting errors.

With a Next app I have the script setup like this, since next build internally runs the linter the build fails if theres any warnings.

1
 "build": "next build",
2
 "lint": "next lint --max-warnings 0",

Now with a Vite app similarly setup like this, if I run lint directly it fails, but if I run build the vite doesn't call lint by default:

1
"build": "tsc && vite build",
2
"lint": "eslint \"src/**/*.ts\"  --max-warnings 0"

Is the correct way to fix this to add lint to build script (make it act like the Next apps):

1
"build": "tsc &&  pnpm lint && vite build",

Or modify my turbo.json to always run lint before build (feels unnecessary for the Next apps that already do it):

1
"tasks": {
2
    "build": {
3
        "dependsOn": ["lint", "^build"],
4
    }
5
}

Or maybe target just the specific Vite app (seems funky to have a #lint run in the #build pipeline):

1
"tasks": {
2
    "viteapp#build": {
3
      "dependsOn": ["viteapp#lint"] 
4
    }
5
    "build": {
6
        "dependsOn": ["^build"],
7
    }
8
}

Or disabling the built in Next build linting so turbo can handle it all:

1
const nextConfig = {
2
  eslint: {
3
    // Handled by Turbo
4
    ignoreDuringBuilds: true,
5
  },
6
};
7
export default nextConfig;

Or is there a better way to handle this? I'm aware I could probably vite-plugin-eslint to force the build to run the lint but I wanted to see if there way a more correct "turbo way".