google / google/pprof-nodejs

Consider a maintained replacement for the deep-copy test dependency

Aperta Adatta ai principianti
#359 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
JavaScript
Stelle
291
Fork
37
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

`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.

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia da package.json e dal file di lock, quindi esamina ts/test/test-heap-profiler.ts per confermare l’uso esistente di require('deep-copy'). Se l’alias viene approvato, aggiorna la dichiarazione della dipendenza e il file di lock senza modificare l’import del test, quindi esegui la suite di test heap-profiler e verifica che i test passino su Node >=18.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
javascript, node.js
Ambito
testing-qa
Tipo di issue
Refactoring
Difficoltà
2/5
Tempo stimato
1-3 ore
Stato di attività
Attiva
Chiarezza
Abbastanza chiara
Idoneità per principianti
72/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.