Skip to content

Commit

Permalink
chore(deps): remove 2 unnecessary dev deps (#7025)
Browse files Browse the repository at this point in the history
  • Loading branch information
serhalp authored Feb 6, 2025
1 parent f74c6c6 commit 6d5af3f
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 73 deletions.
65 changes: 0 additions & 65 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 0 additions & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -182,7 +182,6 @@
"@netlify/eslint-config-node": "7.0.1",
"@netlify/functions": "3.0.0",
"@sindresorhus/slugify": "2.2.1",
"@types/fs-extra": "11.0.4",
"@types/inquirer": "9.0.7",
"@types/jsonwebtoken": "9.0.8",
"@types/lodash": "4.17.15",
Expand All @@ -198,7 +197,6 @@
"eslint-plugin-sort-destructure-keys": "2.0.0",
"eslint-plugin-workspace": "file:./tools/lint-rules",
"form-data": "4.0.1",
"fs-extra": "11.3.0",
"husky": "9.1.7",
"is-ci": "4.1.0",
"nock": "14.0.0",
Expand Down
7 changes: 5 additions & 2 deletions tests/integration/utils/fixture.ts
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
import { cp } from 'fs/promises'
import { join } from 'path'
import { fileURLToPath } from 'url'

import type { NodeOptions } from 'execa'
import { copy } from 'fs-extra'
import { temporaryDirectory } from 'tempy'
import { afterAll, afterEach, beforeAll, beforeEach, describe } from 'vitest'

Expand Down Expand Up @@ -75,7 +75,10 @@ export class Fixture {
static async create(fixturePath: string, options?: FixtureSettings): Promise<Fixture> {
const fixture = new Fixture(fixturePath, temporaryDirectory(), options)

await copy(join(FIXTURES_DIRECTORY, fixturePath), fixture.directory)
await cp(join(FIXTURES_DIRECTORY, fixturePath), fixture.directory, {
recursive: true,
verbatimSymlinks: true,
})

return fixture
}
Expand Down
7 changes: 3 additions & 4 deletions tests/unit/utils/get-global-config.test.js
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
import { existsSync } from 'fs'
import { mkdir, readFile, rm, writeFile } from 'fs/promises'
import { cp, mkdir, readFile, rm, writeFile } from 'fs/promises'
import os from 'os'
import { join } from 'path'

import { copy } from 'fs-extra'
import { afterAll, beforeAll, beforeEach, expect, test } from 'vitest'

import { getLegacyPathInHome, getPathInHome } from '../../../dist/lib/settings.js'
Expand All @@ -20,7 +19,7 @@ const tmpConfigBackupPath = join(os.tmpdir(), `netlify-config-backup-${Date.now(
beforeAll(async () => {
// backup current user config directory
if (existsSync(configPath)) {
await copy(configPath, tmpConfigBackupPath)
await cp(configPath, tmpConfigBackupPath, { recursive: true })
}
})

Expand All @@ -34,7 +33,7 @@ afterAll(async () => {
// Restore user config directory if exists
if (existsSync(tmpConfigBackupPath)) {
await mkdir(configPath)
await copy(tmpConfigBackupPath, configPath)
await cp(tmpConfigBackupPath, configPath, { recursive: true })
// Remove tmp backup
await rm(tmpConfigBackupPath, { force: true, recursive: true })
}
Expand Down

2 comments on commit 6d5af3f

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📊 Benchmark results

  • Dependency count: 1,192
  • Package size: 306 MB
  • Number of ts-expect-error directives: 801

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📊 Benchmark results

  • Dependency count: 1,192
  • Package size: 306 MB
  • Number of ts-expect-error directives: 801

Please sign in to comment.