fix: remove max-len linting bypasses

wraithgar · wraithgar · commit 526473bf1f2f · 2025-01-21T08:46:51.000-08:00
The newest @npmcli/eslint-config allows these to be long
diff --git a/lib/normalize.js b/lib/normalize.js
@@ -487,7 +487,6 @@ const normalize = async (pkg, { strict, steps, root, changes, allowLegacyCase })
   // Some steps are isolated so we can do a limited subset of these in `fix`
   if (steps.includes('fixRepositoryField') || steps.includes('normalizeData')) {
     if (data.repositories) {
-      /* eslint-disable-next-line max-len */
       changes?.push(`"repository" was set to the first entry in "repositories" (${data.repository})`)
       data.repository = data.repositories[0]
     }
diff --git a/test/prepare.js b/test/prepare.js
@@ -617,7 +617,6 @@ for (const [name, testPrepare] of Object.entries(testMethods)) {
         t.has(content, { type: undefined })
       })
 
-      // eslint-disable-next-line max-len
       // https://nodejs.org/api/esm.html#esm_writing_dual_packages_while_avoiding_or_minimizing_hazards
 
       t.skip('handles esm modules', async t => {

Original file line number	Diff line number	Diff line change
`@@ -487,7 +487,6 @@ const normalize = async (pkg, { strict, steps, root, changes, allowLegacyCase })`
`487`	`487`	// Some steps are isolated so we can do a limited subset of these in `fix`
`488`	`488`	`if (steps.includes('fixRepositoryField') \|\| steps.includes('normalizeData')) {`
`489`	`489`	`if (data.repositories) {`
`490`		`- /* eslint-disable-next-line max-len */`
`491`	`490`	changes?.push(`"repository" was set to the first entry in "repositories" (${data.repository})`)
`492`	`491`	`data.repository = data.repositories[0]`
`493`	`492`	`}`