Create Valgrind run of tests

Ouverte
  • Quelque chose ne va pas ?
#585 0 commentaires 1 réaction 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
4/5
Temps estimé
3-5 jours
Accessibilité débutants
38/100
Type d'issue
Fonctionnalité
Clarté
À clarifier
Activité
À l'abandon
Stack technique
cpp, nodejs
Domaine
testing-qa

Piste de recherche

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.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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.

Langage dominant
C++
Étoiles
2.6k
Forks
598
Aucune PR mergée en 30 j

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de nodejs/node-addon-examples

Toutes les issues de nodejs/node-addon-examples

Issues similaires

Plus d'issues C++

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.