Create Valgrind run of tests

Offen
#585 0 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Anfängerfreundlichkeit
38/100
Issue-Typ
Feature
Klarheit
Muss geklärt werden
Aktivitätsstatus
Veraltet
Tech-Stack
cpp, nodejs
Bereich
testing-qa

Rechercherichtung

Start by reviewing issue #580 and the node-api team discussion, then locate the repository's test runner and configuration for adding a Valgrind run. Done means the existing tests execute under Valgrind and memory leaks are reported clearly.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

enhancement

Based on discussion in the node-api team meeting today it would be a good idea to add a run of the tests under valgrind to ensure we don't have any memory leaks. This was from the discussion of #580 which we agreed reported a memory leak.

Vorherrschende Sprache
C++
Sterne
2.6k
Forks
598
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus nodejs/node-addon-examples

Alle Issues in nodejs/node-addon-examples

Ähnliche Issues

Weitere Issues zu C++

Neue Issues direkt in Ihr Postfach

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