google / google/pprof-nodejs

Consider a maintained replacement for the deep-copy test dependency

Offen Anfängerfreundlich
#359 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
JavaScript
Sterne
291
Forks
37
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

`package.json` currently declares `deep-copy@^1.4.2` as a direct dev dependency. `ts/test/test-heap-profiler.ts` loads it with `require('deep-copy')` and uses it to clone the plain profile fixtures in the heap-profiler tests.

Would the maintainers approve replacing that declaration with an npm alias?

```json
"deep-copy": "npm:@stackline/deep-copy@1.0.0"
```

The alias would preserve the existing import key and callable CommonJS API, so the test source would not need to change. The replacement supports the repository's Node `>=18` range and has no production dependencies. I maintain `@stackline/deep-copy`; its package and compatibility documentation are available at https://www.npmjs.com/package/@stackline/deep-copy and https://alexandro.net/docs/vanilla/deep-copy/contract.html.

If this direction is acceptable, I can prepare the focused manifest/lockfile change and run the heap-profiler test suite. Keeping the current version or choosing another maintained compatible implementation are also reasonable alternatives if the project prefers them.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne mit package.json und der Lockdatei und prüfe anschließend ts/test/test-heap-profiler.ts, um die bestehende Verwendung von require('deep-copy') zu bestätigen. Wenn der Alias genehmigt ist, aktualisiere die Abhängigkeitsdeklaration und die Lockdatei, ohne den Testimport zu ändern, führe anschließend die heap-profiler-Testsuite aus und verifiziere, dass die Tests unter Node >=18 bestehen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript, node.js
Bereich
testing-qa
Issue-Typ
Refactoring
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Aktiv
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
72/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.