openPMD/openPMD-api

Namespace Per Dir

Ouverte

#82 ouverte le 19 févr. 2018

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)C++ (57 forks)auto 404
api: breakinggood first issuehelp wanted

Métriques du dépôt

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

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)

Guide contributeur

Direction de recherche
Inspectez la structure des répertoires sous include/openPMD/, ajoutez des déclarations d'espace de noms correspondant au nom de chaque dossier à tous les fichiers d'en tête/source, et mettez à jour le code qui fait référence à ces classes/fonctions pour utiliser les nouveaux espaces de noms.
Stack technique
cpp
Domaine
backend
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.
Une demi journée
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++Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75