nodejs/llnode

no coverage for "v8 nodeinfo"

Aperta

#423 aperta il 1 ott 2022

Vedi su GitHubGuida contributor
 (4 commenti) (0 reazioni) (0 assegnatari)C++ (101 fork)github user discovery
good first issue

Metriche repository

Star
 (1167 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

According to the coverage report there are no tests for v8 nodeinfo. Two options:

  1. Remove it. It is a couple hundred lines of code, and how often do you get a coredump where you don't know the version of Node.js you were running?
  2. Add a test for it.

Guida contributor

Direzione di ricerca
Esamina l'infrastruttura di test esistente e trova come aggiungere un test per il comando 'v8 nodeinfo'. Considera di scrivere un test o di raggiungere un consenso per rimuovere il codice.
Tech stack
nodejs
Dominio
backendtooling
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Node.jslldb
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60