rapidsai/cudf

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

Open

#12.112 aberto em 9 de nov. de 2022

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)C++ (735 forks)batch import
breakingfeature requestgood first issueimprovementlibcudf

Métricas do repositório

Stars
 (6.000 stars)
Métricas de merge de PR
 (Mesclagem média 17d 21h) (230 fundiu PRs em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Identifique todos os pares de cabeçalhos públicos e de detalhe onde os nomes diferem (por exemplo, cudf/copying.hpp vs cudf/detail/copy.hpp). Para cada par, decida o nome correto (provavelmente o nome do cabeçalho público) e renomeie o arquivo de cabeçalho de detalhe de acordo. Atualize todas as instruções include que referenciam o nome antigo do cabeçalho de detalhe. Verifique quaisquer dependências em scripts de compilação ou documentação.
Pilha de tecnologia
cpp
Domain
backenddata
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitC++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
90