rapidsai/cudf

[FEA] Public header and detail header files should have the same names

Open

#12.112 aperta il 9 nov 2022

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)C++ (735 fork)batch import
breakingfeature requestgood first issueimprovementlibcudf

Metriche repository

Star
 (6000 star)
Metriche merge PR
 (Merge medio 17g 21h) (230 PR mergiate in 30 g)

Descrizione

As the title said, the public and detail headers should have the same names. Currently there are some inconsistencies between them. For example:

  • Public header: cudf/copying.hpp,
  • Detail header: cudf/detail/copy.hpp.

We should change the file names to make them consistent.

Guida contributor

Direzione di ricerca
Identifica tutte le coppie di intestazioni pubbliche e di dettaglio in cui i nomi differiscono (ad es. cudf/copying.hpp vs cudf/detail/copy.hpp). Per ogni coppia, decidi il nome corretto (probabilmente quello dell'intestazione pubblica) e rinomina di conseguenza il file di intestazione di dettaglio. Aggiorna tutte le istruzioni include che fanno riferimento al vecchio nome dell'intestazione di dettaglio. Controlla eventuali dipendenze negli script di compilazione o nella documentazione.
Tech stack
cpp
Dominio
backenddata
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
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
GitC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90