diff --git a/.github/workflows/lint-404s.yml b/.github/workflows/lint-404s.yml index 5559794db14e7..1015b88dd6971 100644 --- a/.github/workflows/lint-404s.yml +++ b/.github/workflows/lint-404s.yml @@ -32,8 +32,13 @@ jobs: - uses: actions/cache@v4 id: cache with: - path: ${{ github.workspace }}/node_modules + path: | + ${{ github.workspace }}/node_modules + ${{ github.workspace }}/.next/cache + ${{ github.workspace }}/.eslintcache key: node-${{ runner.os }}-${{ hashFiles('**/yarn.lock') }} + restore-keys: | + node-${{ runner.os }}- - run: yarn install --frozen-lockfile if: steps.cache.outputs.cache-hit != 'true' diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 01bc4eb9af85c..fcf3fd0ab47c5 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -29,18 +29,23 @@ jobs: - uses: actions/cache@v4 id: cache with: - path: ${{ github.workspace }}/node_modules + path: | + ${{ github.workspace }}/node_modules + ${{ github.workspace }}/.next/cache + ${{ github.workspace }}/.eslintcache key: node-${{ runner.os }}-${{ hashFiles('**/yarn.lock') }} + restore-keys: | + node-${{ runner.os }}- + - run: yarn install --frozen-lockfile if: steps.cache.outputs.cache-hit != 'true' # Additional checks - run: yarn lint:ts - - run: yarn lint:docs # Run automatic fixes (run prettier apart from eslint to also fix mdx files) - run: yarn lint:prettier:fix - - run: yarn lint:eslint:fix + - run: yarn lint:eslint:fix --cache # Check (and error) for dirty working tree for forks # Reason being we need a different token to auto commit changes and @@ -69,8 +74,14 @@ jobs: - uses: actions/cache@v4 id: cache with: - path: ${{ github.workspace }}/node_modules + path: | + ${{ github.workspace }}/node_modules + ${{ github.workspace }}/.next/cache + ${{ github.workspace }}/.eslintcache key: node-${{ runner.os }}-${{ hashFiles('**/yarn.lock') }} + restore-keys: | + node-${{ runner.os }}- + - run: yarn install --frozen-lockfile if: steps.cache.outputs.cache-hit != 'true' - name: Run Tests diff --git a/bin/lint-docs.ts b/bin/lint-docs.ts deleted file mode 100755 index 2843b3adf5429..0000000000000 --- a/bin/lint-docs.ts +++ /dev/null @@ -1,6 +0,0 @@ -#!/usr/bin/env ts-node - -// This file used to lint for wizard errors. Wizard docs have been removed -// (possibly permanently). If they come back, this should be ported from the -// Gatsby version. See: -// https://github.com/getsentry/sentry-docs/blob/405170000995bd8d98bd155f4e8d38ce4c19275a/bin/lint-docs.ts diff --git a/package.json b/package.json index 7de8924a8a6db..1b08d54f5a5a2 100644 --- a/package.json +++ b/package.json @@ -25,7 +25,6 @@ "start": "next start", "lint": "next lint", "lint:ts": "tsc --skipLibCheck", - "lint:docs": "bin/lint-docs.ts", "lint:eslint": "eslint \"{src,app,scripts}/**/*.{ts,tsx,js,jsx}\"", "lint:eslint:fix": "eslint --fix \"{src,app,scripts}/**/*.{ts,tsx,js,jsx}\"", "lint:prettier": "prettier --check \"./{src,app,scripts}/**/*.{md,mdx,ts,tsx,js,jsx,mjs}\"",