Use ES6 import instead of require for dotenv

mraible · mraible · commit 5d0b48437b66 · 2025-11-05T12:40:24.000-07:00
Addresses PR feedback from Javier to avoid mixing import/require syntax
diff --git a/e2e/playwright.config.ts b/e2e/playwright.config.ts
@@ -1,8 +1,9 @@
 import { defineConfig, devices } from '@playwright/test';
 import { AuthFile } from './constants/AuthFile';
+import dotenv from 'dotenv';
 
 if (!process.env.CI) {
-  require("dotenv").config({ path: ".env", quiet: true });
+  dotenv.config({ path: ".env", quiet: true });
 }
 
 export default defineConfig({

Original file line number	Diff line number	Diff line change
`@@ -1,8 +1,9 @@`
`1`	`1`	`import { defineConfig, devices } from '@playwright/test';`
`2`	`2`	`import { AuthFile } from './constants/AuthFile';`
	`3`	`+import dotenv from 'dotenv';`
`3`	`4`
`4`	`5`	`if (!process.env.CI) {`
`5`		`- require("dotenv").config({ path: ".env", quiet: true });`
	`6`	`+ dotenv.config({ path: ".env", quiet: true });`
`6`	`7`	`}`
`7`	`8`
`8`	`9`	`export default defineConfig({`