nim-works/nimskull

Include valgrind as part of CI

Aperta

#588 aperta il 17 mar 2023

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)Nim (39 fork)auto 404
cicompilergood first issuehelp wanted

Metriche repository

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

Descrizione

In the past while it has become evident that there are a number of fundamnetal issues with our memory management implementations (refc, arc, and orc) that lead them to be unsafe, especially in conjunction with threading. Before even getting to complex multi-threading scenarios we can eliminate lots of issues by addressing valgrind's findings.

TODO:

  • Enable valgrind for tests in CI; a number of tests already support this
  • tackle a few quick wins to get some momentum

Guida contributor

Direzione di ricerca
Ispeziona i file di configurazione CI (ad es. .github/workflows) per comprendere l'attuale configurazione dei test, quindi identifica i test che già supportano valgrind come menzionato nell'elenco TODO e abilita valgrind per tutti i test aggiungendo un passaggio valgrind o utilizzando il supporto esistente.
Tech stack
Nessuno
Dominio
devops
Tipo issue
DevOps
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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
CI/CDvalgrind
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65