fix linting

kentcdodds · kentcdodds · commit 4cab84495470 · 2026-01-14T20:46:12.000-07:00
diff --git a/epicshop/fix-watch.js b/epicshop/fix-watch.js
@@ -26,10 +26,10 @@ const debouncedRun = debounce(run, 200)
 
 // Add event listeners.
 watcher
-	.on('addDir', (path) => {
+	.on('addDir', (_path) => {
 		debouncedRun()
 	})
-	.on('unlinkDir', (path) => {
+	.on('unlinkDir', (_path) => {
 		debouncedRun()
 	})
 	.on('error', (error) => console.log(`Watcher error: ${error}`))
diff --git a/epicshop/fix.js b/epicshop/fix.js
@@ -1,14 +1,14 @@
 // This should run by node without any dependencies
 // because you may need to run it without deps.
 
-import cp from 'node:child_process'
 import fs from 'node:fs'
 import path from 'node:path'
 import { fileURLToPath } from 'node:url'
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url))
 const here = (...p) => path.join(__dirname, ...p)
 const VERBOSE = false
+// eslint-disable-next-line
 const logVerbose = (...args) => (VERBOSE ? console.log(...args) : undefined)
 
 const workshopRoot = here('..')
@@ -78,7 +78,7 @@ function exists(p) {
 	try {
 		fs.statSync(p)
 		return true
-	} catch (error) {
+	} catch {
 		return false
 	}
 }
diff --git a/exercises/01.classes/01.problem.class-basics/index.ts b/exercises/01.classes/01.problem.class-basics/index.ts
@@ -26,7 +26,7 @@
 // sampleCart.addItem(sampleLaptop)
 // sampleCart.addItem(sampleMouse)
 // console.log(
-// 	'Results JSON:',
+// 	'Results:',
 // 	JSON.stringify({
 // 		product: {
 // 			name: sampleLaptop.name,
diff --git a/exercises/01.classes/01.solution.class-basics/index.test.ts b/exercises/01.classes/01.solution.class-basics/index.test.ts
@@ -3,12 +3,10 @@ import { execSync } from 'node:child_process'
 import { test } from 'node:test'
 
 const output = execSync('npm start --silent', { encoding: 'utf8' })
-const jsonLine = output
-	.split('\n')
-	.find((line) => line.startsWith('Results JSON:'))
-assert.ok(jsonLine, '🚨 Missing "Results JSON:" output line')
+const jsonLine = output.split('\n').find((line) => line.startsWith('Results:'))
+assert.ok(jsonLine, '🚨 Missing "Results:" output line')
 const { product, cart, emptyCart } = JSON.parse(
-	jsonLine.replace('Results JSON:', '').trim(),
+	jsonLine.replace('Results:', '').trim(),
 )
 
 await test('Product class should create instances with name and price', () => {
diff --git a/exercises/01.classes/01.solution.class-basics/index.ts b/exercises/01.classes/01.solution.class-basics/index.ts
@@ -50,7 +50,7 @@ const afterFirstAddCount = sampleCart.items.length
 sampleCart.addItem(sampleMouse)
 
 console.log(
-	'Results JSON:',
+	'Results:',
 	JSON.stringify({
 		product: {
 			name: sampleLaptop.name,
diff --git a/exercises/01.classes/02.problem.constructors/index.ts b/exercises/01.classes/02.problem.constructors/index.ts
@@ -31,7 +31,7 @@
 // sampleAccount.deposit(100)
 // sampleAccount.deposit(50)
 // console.log(
-// 	'Results JSON:',
+// 	'Results:',
 // 	JSON.stringify({
 // 		user: { name: user.name, email: user.email, role: user.role },
 // 		admin: { name: admin.name, email: admin.email, role: admin.role },
diff --git a/exercises/01.classes/02.solution.constructors/index.test.ts b/exercises/01.classes/02.solution.constructors/index.test.ts
@@ -3,12 +3,10 @@ import { execSync } from 'node:child_process'
 import { test } from 'node:test'
 
 const output = execSync('npm start --silent', { encoding: 'utf8' })
-const jsonLine = output
-	.split('\n')
-	.find((line) => line.startsWith('Results JSON:'))
-assert.ok(jsonLine, '🚨 Missing "Results JSON:" output line')
+const jsonLine = output.split('\n').find((line) => line.startsWith('Results:'))
+assert.ok(jsonLine, '🚨 Missing "Results:" output line')
 const { user, admin, account, config, customConfig } = JSON.parse(
-	jsonLine.replace('Results JSON:', '').trim(),
+	jsonLine.replace('Results:', '').trim(),
 )
 
 await test('User constructor should set name, email, and default role', () => {
diff --git a/exercises/01.classes/02.solution.constructors/index.ts b/exercises/01.classes/02.solution.constructors/index.ts
@@ -66,7 +66,7 @@ const balanceAfterFirstDeposit = sampleAccount.getBalance()
 sampleAccount.deposit(50)
 
 console.log(
-	'Results JSON:',
+	'Results:',
 	JSON.stringify({
 		user: { name: user.name, email: user.email, role: user.role },
 		admin: { name: admin.name, email: admin.email, role: admin.role },
diff --git a/exercises/02.interfaces-and-classes/01.problem.implementing-interfaces/index.ts b/exercises/02.interfaces-and-classes/01.problem.implementing-interfaces/index.ts
@@ -21,7 +21,7 @@
 
 // 🐨 When you're done, uncomment this:
 // console.log(
-// 	'Results JSON:',
+// 	'Results:',
 // 	JSON.stringify({
 // 		creditCard: {
 // 			cardNumber: creditCard.cardNumber,
diff --git a/exercises/02.interfaces-and-classes/01.solution.implementing-interfaces/index.test.ts b/exercises/02.interfaces-and-classes/01.solution.implementing-interfaces/index.test.ts
@@ -3,12 +3,10 @@ import { execSync } from 'node:child_process'
 import { test } from 'node:test'
 
 const output = execSync('npm start --silent', { encoding: 'utf8' })
-const jsonLine = output
-	.split('\n')
-	.find((line) => line.startsWith('Results JSON:'))
-assert.ok(jsonLine, '🚨 Missing "Results JSON:" output line')
+const jsonLine = output.split('\n').find((line) => line.startsWith('Results:'))
+assert.ok(jsonLine, '🚨 Missing "Results:" output line')
 const { creditCard, paypal } = JSON.parse(
-	jsonLine.replace('Results JSON:', '').trim(),
+	jsonLine.replace('Results:', '').trim(),
 )
 
 await test('CreditCard should implement PaymentMethod interface', () => {
diff --git a/exercises/02.interfaces-and-classes/01.solution.implementing-interfaces/index.ts b/exercises/02.interfaces-and-classes/01.solution.implementing-interfaces/index.ts
@@ -35,7 +35,7 @@ console.log(creditCard.pay(100))
 console.log(paypal.pay(50))
 
 console.log(
-	'Results JSON:',
+	'Results:',
 	JSON.stringify({
 		creditCard: {
 			cardNumber: creditCard.cardNumber,
diff --git a/exercises/02.interfaces-and-classes/02.problem.programming-to-abstractions/index.ts b/exercises/02.interfaces-and-classes/02.problem.programming-to-abstractions/index.ts
@@ -28,7 +28,7 @@
 // const testCard = new CreditCard('1111-2222-3333-4444')
 // const testPayPal = new PayPal('test@example.com')
 // console.log(
-// 	'Results JSON:',
+// 	'Results:',
 // 	JSON.stringify({
 // 		creditCard: processPayment(creditCard, 100),
 // 		paypal: processPayment(paypal, 50),
diff --git a/exercises/02.interfaces-and-classes/02.solution.programming-to-abstractions/index.test.ts b/exercises/02.interfaces-and-classes/02.solution.programming-to-abstractions/index.test.ts
@@ -3,12 +3,10 @@ import { execSync } from 'node:child_process'
 import { test } from 'node:test'
 
 const output = execSync('npm start --silent', { encoding: 'utf8' })
-const jsonLine = output
-	.split('\n')
-	.find((line) => line.startsWith('Results JSON:'))
-assert.ok(jsonLine, '🚨 Missing "Results JSON:" output line')
+const jsonLine = output.split('\n').find((line) => line.startsWith('Results:'))
+assert.ok(jsonLine, '🚨 Missing "Results:" output line')
 const { creditCard, paypal, testCard, testPayPal } = JSON.parse(
-	jsonLine.replace('Results JSON:', '').trim(),
+	jsonLine.replace('Results:', '').trim(),
 )
 
 await test('processPayment should work with CreditCard', () => {
diff --git a/exercises/02.interfaces-and-classes/02.solution.programming-to-abstractions/index.ts b/exercises/02.interfaces-and-classes/02.solution.programming-to-abstractions/index.ts
@@ -43,7 +43,7 @@ const testCard = new CreditCard('1111-2222-3333-4444')
 const testPayPal = new PayPal('test@example.com')
 
 console.log(
-	'Results JSON:',
+	'Results:',
 	JSON.stringify({
 		creditCard: processPayment(creditCard, 100),
 		paypal: processPayment(paypal, 50),
diff --git a/exercises/03.inheritance/01.problem.extends/index.ts b/exercises/03.inheritance/01.problem.extends/index.ts
@@ -26,7 +26,7 @@
 // const sampleCircle = new Circle('red', 5)
 // const sampleRectangle = new Rectangle('blue', 10, 20)
 // console.log(
-// 	'Results JSON:',
+// 	'Results:',
 // 	JSON.stringify({
 // 		circle: { color: sampleCircle.color, radius: sampleCircle.radius },
 // 		rectangle: {
diff --git a/exercises/03.inheritance/01.solution.extends/index.test.ts b/exercises/03.inheritance/01.solution.extends/index.test.ts
@@ -3,12 +3,10 @@ import { execSync } from 'node:child_process'
 import { test } from 'node:test'
 
 const output = execSync('npm start --silent', { encoding: 'utf8' })
-const jsonLine = output
-	.split('\n')
-	.find((line) => line.startsWith('Results JSON:'))
-assert.ok(jsonLine, '🚨 Missing "Results JSON:" output line')
+const jsonLine = output.split('\n').find((line) => line.startsWith('Results:'))
+assert.ok(jsonLine, '🚨 Missing "Results:" output line')
 const { circle, rectangle } = JSON.parse(
-	jsonLine.replace('Results JSON:', '').trim(),
+	jsonLine.replace('Results:', '').trim(),
 )
 
 await test('Circle should inherit color from Shape', () => {
diff --git a/exercises/03.inheritance/01.solution.extends/index.ts b/exercises/03.inheritance/01.solution.extends/index.ts
@@ -40,7 +40,7 @@ const sampleCircle = new Circle('red', 5)
 const sampleRectangle = new Rectangle('blue', 10, 20)
 
 console.log(
-	'Results JSON:',
+	'Results:',
 	JSON.stringify({
 		circle: { color: sampleCircle.color, radius: sampleCircle.radius },
 		rectangle: {
diff --git a/exercises/03.inheritance/02.problem.method-overriding/index.ts b/exercises/03.inheritance/02.problem.method-overriding/index.ts
@@ -28,7 +28,7 @@
 // const circleArea = sampleCircle.getArea()
 // const rectangleArea = sampleRectangle.getArea()
 // console.log(
-// 	'Results JSON:',
+// 	'Results:',
 // 	JSON.stringify({
 // 		shapeArea: baseShape.getArea(),
 // 		circleArea,
diff --git a/exercises/03.inheritance/02.solution.method-overriding/index.test.ts b/exercises/03.inheritance/02.solution.method-overriding/index.test.ts
@@ -3,12 +3,10 @@ import { execSync } from 'node:child_process'
 import { test } from 'node:test'
 
 const output = execSync('npm start --silent', { encoding: 'utf8' })
-const jsonLine = output
-	.split('\n')
-	.find((line) => line.startsWith('Results JSON:'))
-assert.ok(jsonLine, '🚨 Missing "Results JSON:" output line')
+const jsonLine = output.split('\n').find((line) => line.startsWith('Results:'))
+assert.ok(jsonLine, '🚨 Missing "Results:" output line')
 const { shapeArea, circleArea, rectangleArea } = JSON.parse(
-	jsonLine.replace('Results JSON:', '').trim(),
+	jsonLine.replace('Results:', '').trim(),
 )
 
 await test('Shape getArea should return 0', () => {
diff --git a/exercises/03.inheritance/02.solution.method-overriding/index.ts b/exercises/03.inheritance/02.solution.method-overriding/index.ts
@@ -56,7 +56,7 @@ const circleArea = sampleCircle.getArea()
 const rectangleArea = sampleRectangle.getArea()
 
 console.log(
-	'Results JSON:',
+	'Results:',
 	JSON.stringify({
 		shapeArea: baseShape.getArea(),
 		circleArea,
diff --git a/exercises/04.polymorphism/01.problem.substitutability/index.ts b/exercises/04.polymorphism/01.problem.substitutability/index.ts
@@ -26,7 +26,7 @@
 // const base = new MediaFile('file.mp3')
 // const basePlayer = new MediaPlayer()
 // console.log(
-// 	'Results JSON:',
+// 	'Results:',
 // 	JSON.stringify({
 // 		base: basePlayer.playFile(base),
 // 		audio: basePlayer.playFile(audio),
diff --git a/exercises/04.polymorphism/01.solution.substitutability/index.test.ts b/exercises/04.polymorphism/01.solution.substitutability/index.test.ts
@@ -3,12 +3,10 @@ import { execSync } from 'node:child_process'
 import { test } from 'node:test'
 
 const output = execSync('npm start --silent', { encoding: 'utf8' })
-const jsonLine = output
-	.split('\n')
-	.find((line) => line.startsWith('Results JSON:'))
-assert.ok(jsonLine, '🚨 Missing "Results JSON:" output line')
+const jsonLine = output.split('\n').find((line) => line.startsWith('Results:'))
+assert.ok(jsonLine, '🚨 Missing "Results:" output line')
 const { base, audio, video, substitutable } = JSON.parse(
-	jsonLine.replace('Results JSON:', '').trim(),
+	jsonLine.replace('Results:', '').trim(),
 )
 
 await test('MediaPlayer should accept MediaFile instances', () => {
diff --git a/exercises/04.polymorphism/01.solution.substitutability/index.ts b/exercises/04.polymorphism/01.solution.substitutability/index.ts
@@ -43,7 +43,7 @@ const base = new MediaFile('file.mp3')
 const basePlayer = new MediaPlayer()
 
 console.log(
-	'Results JSON:',
+	'Results:',
 	JSON.stringify({
 		base: basePlayer.playFile(base),
 		audio: basePlayer.playFile(audio),
diff --git a/exercises/05.composition-vs-inheritance/01.problem.when-to-use/index.ts b/exercises/05.composition-vs-inheritance/01.problem.when-to-use/index.ts
@@ -29,7 +29,7 @@
 
 // 🐨 When you're done, uncomment this:
 // console.log(
-// 	'Results JSON:',
+// 	'Results:',
 // 	JSON.stringify({
 // 		fileLoggerIsLogger: fileLogger instanceof Logger,
 // 		consoleLoggerIsLogger: consoleLogger instanceof Logger,
diff --git a/exercises/05.composition-vs-inheritance/01.solution.when-to-use/index.test.ts b/exercises/05.composition-vs-inheritance/01.solution.when-to-use/index.test.ts
@@ -3,10 +3,8 @@ import { execSync } from 'node:child_process'
 import { test } from 'node:test'
 
 const output = execSync('npm start --silent', { encoding: 'utf8' })
-const jsonLine = output
-	.split('\n')
-	.find((line) => line.startsWith('Results JSON:'))
-assert.ok(jsonLine, '🚨 Missing "Results JSON:" output line')
+const jsonLine = output.split('\n').find((line) => line.startsWith('Results:'))
+assert.ok(jsonLine, '🚨 Missing "Results:" output line')
 const {
 	fileLoggerIsLogger,
 	consoleLoggerIsLogger,
@@ -15,7 +13,7 @@ const {
 	sendEmailExists,
 	emailService1IsInstance,
 	emailService2IsInstance,
-} = JSON.parse(jsonLine.replace('Results JSON:', '').trim())
+} = JSON.parse(jsonLine.replace('Results:', '').trim())
 
 await test('FileLogger should extend Logger (inheritance)', () => {
 	assert.ok(
diff --git a/exercises/05.composition-vs-inheritance/01.solution.when-to-use/index.ts b/exercises/05.composition-vs-inheritance/01.solution.when-to-use/index.ts
@@ -49,7 +49,7 @@ emailService1.sendEmail('user@example.com', 'Welcome')
 emailService2.sendEmail('admin@example.com', 'Alert')
 
 console.log(
-	'Results JSON:',
+	'Results:',
 	JSON.stringify({
 		fileLoggerIsLogger: fileLogger instanceof Logger,
 		consoleLoggerIsLogger: consoleLogger instanceof Logger,
