openPMD/openPMD-api

Namespace Per Dir

Aberta

#82 aberto em 19 de fev. de 2018

Ver no GitHubGuia do colaborador
 (0 comentário) (0 reação) (0 responsável)C++ (57 forks)auto 404
api: breakinggood first issuehelp wanted

Métricas do repositório

Stars
 (160 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Follow-up to #81.

Each directory inside include/openPMD/ should define everything in a namespace with the same name as the dir.

  • backend/ (openPMD::backend)
  • IO/ (openPMD::IO)
    • ADIOS/ (openPMD::IO::ADIOS)
    • HDF5/ (openPMD::IO::HDF5)

Guia do colaborador

Direção de pesquisa
Inspecione a estrutura de diretórios sob include/openPMD/, adicione declarações de namespace correspondentes ao nome de cada pasta a todos os arquivos de cabeçalho/fonte e atualize o código que faz referência a essas classes/funções para usar os novos namespaces.
Pilha de tecnologia
cpp
Domain
backend
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.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
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
C++Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75