nim-works/nimskull

Include valgrind as part of CI

Ouverte

#588 ouverte le 17 mars 2023

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)Nim (39 forks)auto 404
cicompilergood first issuehelp wanted

Métriques du dépôt

Stars
 (346 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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

Guide contributeur

Direction de recherche
Inspectez les fichiers de configuration CI (par exemple .github/workflows) pour comprendre la configuration actuelle des tests, identifiez les tests qui prennent déjà en charge valgrind comme mentionné dans la liste TODO, et activez valgrind pour tous les tests en ajoutant une étape valgrind ou en utilisant le support existant.
Stack technique
Aucun
Domaine
devops
Type d'issue
DevOps
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
CI/CDvalgrind
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65