Duzzuti/passwd-manager

[NTH] Ensure that CMakeLists.txt is following the best practices

Ouverte

#28 ouverte le 8 juin 2023

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)C++ (4 forks)auto 404
enhancementgood first issue

Métriques du dépôt

Stars
 (9 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Since this project is partially aimed at developers improving their C++ and CMake skills I create this issue. The issue should be constantly ongoing and if we realize that some of our practices in CMakeLists.txt are not ideal we should make a PR referencing it.

This is a nice source to learn about the modern best practices for CMake: https://cliutils.gitlab.io/modern-cmake/

  • Difficulty (1-10): 2

  • Location/Files: CMakeLists.txt

Guide contributeur

Direction de recherche
Lisez le guide des bonnes pratiques modernes de CMake au lien fourni et comparez le avec le CMakeLists.txt actuel pour identifier les domaines a ameliorer, puis ouvrez une pull request avec les modifications.
Stack technique
cpp
Domaine
build system
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
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
C++CMake
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80