Duzzuti/passwd-manager

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

Aperta

#28 aperta il 8 giu 2023

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (4 fork)auto 404
enhancementgood first issue

Metriche repository

Star
 (9 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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

Guida contributor

Direzione di ricerca
Leggi la guida alle buone pratiche moderne di CMake al link fornito e confrontala con l'attuale CMakeLists.txt per identificare aree di miglioramento, quindi apri una pull request con le modifiche.
Tech stack
cpp
Dominio
build system
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
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
C++CMake
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80