update eslint and try concurrency

kentcdodds · kentcdodds · commit 7065091be462 · 2025-08-27T13:28:07.000-06:00
diff --git a/.github/workflows/validate.yml b/.github/workflows/validate.yml
@@ -33,8 +33,8 @@ jobs:
         run: npm run typecheck
 
       # TODO: re-enable linting
-      # - name: ⬣ ESLint
-      #   run: npm run lint
+      - name: ⬣ ESLint
+        run: npm run lint
 
       # TODO: get this working again
       # - name: ⬇️ Install Playwright
diff --git a/epicshop/package-lock.json b/epicshop/package-lock.json
diff --git a/epicshop/package.json b/epicshop/package.json
@@ -5,13 +5,13 @@
     "test": "playwright test && node test.js"
   },
   "dependencies": {
-    "@epic-web/config": "^1.16.4",
+    "@epic-web/config": "^1.21.3",
     "@epic-web/workshop-app": "^6.23.4",
     "@epic-web/workshop-utils": "^6.23.4",
     "enquirer": "^2.4.1",
-    "execa": "^9.5.2",
-    "match-sorter": "^8.0.0",
-    "p-limit": "^6.1.0",
+    "execa": "^9.6.0",
+    "match-sorter": "^8.1.0",
+    "p-limit": "^7.1.0",
     "@epic-web/workshop-cli": "^6.23.4"
   }
 }
diff --git a/exercises/04.code-splitting/03.problem.transition/tests/flash.test.ts b/exercises/04.code-splitting/03.problem.transition/tests/flash.test.ts
@@ -43,11 +43,11 @@ test('should still show a pending UI when the globe is not ready', async ({
 	await page.goto('/')
 	await page.waitForLoadState('networkidle')
 
-	await page.route('**/*', (route) => {
+	await page.route('**/*', async (route) => {
 		if (route.request().resourceType() === 'script') {
 			setTimeout(() => route.continue(), 600) // Simulate a slow load time for scripts
 		} else {
-			route.continue()
+			await route.continue()
 		}
 	})
 
diff --git a/exercises/04.code-splitting/03.solution.transition/tests/flash.test.ts b/exercises/04.code-splitting/03.solution.transition/tests/flash.test.ts
@@ -43,11 +43,11 @@ test('should still show a pending UI when the globe is not ready', async ({
 	await page.goto('/')
 	await page.waitForLoadState('networkidle')
 
-	await page.route('**/*', (route) => {
+	await page.route('**/*', async (route) => {
 		if (route.request().resourceType() === 'script') {
 			setTimeout(() => route.continue(), 600) // Simulate a slow load time for scripts
 		} else {
-			route.continue()
+			await route.continue()
 		}
 	})
 
diff --git a/exercises/05.calculations/01.problem.use-memo/tests/use-memo.test.ts b/exercises/05.calculations/01.problem.use-memo/tests/use-memo.test.ts
@@ -24,7 +24,7 @@ export { searchCitiesInstrumented as searchCities }
 		`
 		content = `${content}\n\n${instrumentedCode}`
 
-		route.fulfill({ body: content, headers: response.headers() })
+		await route.fulfill({ body: content, headers: response.headers() })
 	})
 	await page.goto('/')
 	await page.waitForLoadState('networkidle')
diff --git a/exercises/05.calculations/01.solution.use-memo/tests/use-memo.test.ts b/exercises/05.calculations/01.solution.use-memo/tests/use-memo.test.ts
@@ -24,7 +24,7 @@ export { searchCitiesInstrumented as searchCities }
 		`
 		content = `${content}\n\n${instrumentedCode}`
 
-		route.fulfill({ body: content, headers: response.headers() })
+		await route.fulfill({ body: content, headers: response.headers() })
 	})
 	await page.goto('/')
 	await page.waitForLoadState('networkidle')
diff --git a/exercises/06.rerenders/01.problem.memo/tests/memoized.test.ts b/exercises/06.rerenders/01.problem.memo/tests/memoized.test.ts
@@ -74,7 +74,10 @@ test('Only ListItem should not rerender when clicking force rerender', async ({
 			</script>
 		`
 		html = html.replace('<head>', `<head>${scriptToInject}`)
-		route.fulfill({ body: html, headers: { 'content-type': 'text/html' } })
+		await route.fulfill({
+			body: html,
+			headers: { 'content-type': 'text/html' },
+		})
 	})
 
 	await page.goto('/')
diff --git a/exercises/06.rerenders/01.solution.memo/tests/memoized.test.ts b/exercises/06.rerenders/01.solution.memo/tests/memoized.test.ts
@@ -74,7 +74,10 @@ test('Only ListItem should not rerender when clicking force rerender', async ({
 			</script>
 		`
 		html = html.replace('<head>', `<head>${scriptToInject}`)
-		route.fulfill({ body: html, headers: { 'content-type': 'text/html' } })
+		await route.fulfill({
+			body: html,
+			headers: { 'content-type': 'text/html' },
+		})
 	})
 
 	await page.goto('/')
diff --git a/exercises/06.rerenders/02.problem.comparator/tests/memoized.test.ts b/exercises/06.rerenders/02.problem.comparator/tests/memoized.test.ts
@@ -74,7 +74,10 @@ test('Only two ListItems should not rerender when the highlighted item changes',
 			</script>
 		`
 		html = html.replace('<head>', `<head>${scriptToInject}`)
-		route.fulfill({ body: html, headers: { 'content-type': 'text/html' } })
+		await route.fulfill({
+			body: html,
+			headers: { 'content-type': 'text/html' },
+		})
 	})
 
 	await page.goto('/')
diff --git a/exercises/06.rerenders/02.solution.comparator/tests/memoized.test.ts b/exercises/06.rerenders/02.solution.comparator/tests/memoized.test.ts
@@ -74,7 +74,10 @@ test('Only two ListItems should not rerender when the highlighted item changes',
 			</script>
 		`
 		html = html.replace('<head>', `<head>${scriptToInject}`)
-		route.fulfill({ body: html, headers: { 'content-type': 'text/html' } })
+		await route.fulfill({
+			body: html,
+			headers: { 'content-type': 'text/html' },
+		})
 	})
 
 	await page.goto('/')
diff --git a/exercises/06.rerenders/03.problem.primitives/tests/memoized.test.ts b/exercises/06.rerenders/03.problem.primitives/tests/memoized.test.ts
@@ -74,7 +74,10 @@ test('Only two ListItems should not rerender when the highlighted item changes',
 			</script>
 		`
 		html = html.replace('<head>', `<head>${scriptToInject}`)
-		route.fulfill({ body: html, headers: { 'content-type': 'text/html' } })
+		await route.fulfill({
+			body: html,
+			headers: { 'content-type': 'text/html' },
+		})
 	})
 
 	await page.goto('/')
diff --git a/exercises/06.rerenders/03.solution.primitives/tests/memoized.test.ts b/exercises/06.rerenders/03.solution.primitives/tests/memoized.test.ts
@@ -74,7 +74,10 @@ test('Only two ListItems should not rerender when the highlighted item changes',
 			</script>
 		`
 		html = html.replace('<head>', `<head>${scriptToInject}`)
-		route.fulfill({ body: html, headers: { 'content-type': 'text/html' } })
+		await route.fulfill({
+			body: html,
+			headers: { 'content-type': 'text/html' },
+		})
 	})
 
 	await page.goto('/')
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json

Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,7 @@ export { searchCitiesInstrumented as searchCities }`
`24`	`24`	`
`25`	`25`	content = `${content}\n\n${instrumentedCode}`
`26`	`26`
`27`		`- route.fulfill({ body: content, headers: response.headers() })`
	`27`	`+ await route.fulfill({ body: content, headers: response.headers() })`
`28`	`28`	`})`
`29`	`29`	`await page.goto('/')`
`30`	`30`	`await page.waitForLoadState('networkidle')`