Create Valgrind run of tests

Aberta
  • Algo errado?
#585 0 comentários 1 reação 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
4/5
Tempo estimado
3-5 dias
Facilidade para iniciantes
38/100
Tipo de issue
Funcionalidade
Clareza
Precisa de esclarecimento
Status de atividade
Estagnada
Stack de tecnologia
cpp, nodejs
Domínio
testing-qa

Direção de pesquisa

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.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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.

Linguagem predominante
C++
Estrelas
2.6k
Forks
598
Nenhum PR com merge em 30d

Preparar o ambiente

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de nodejs/node-addon-examples

Todas as issues de nodejs/node-addon-examples

Issues semelhantes

Mais issues de C++

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.