apache / apache/datasketches-cpp

Reorganization proposal

Ouverte
#419 6 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
C++
Étoiles
273
Forks
88
Merge moyen
2 j 2 h
PR mergées (30 j)
8

Description

Hi there,

We are consider using datasketches in our C++ environment. The current organization, where there a bunch of folders, each with an `include/` directory, is a bit awkward for integrating into a foreign build system. Projects like [pybind11](https://github.com/pybind/pybind11/) put all of the headers in a single `include/` directory (perhaps with subdirectories underneath). Then there is just a single path to add as an include path.

Would you be open to a reorganization that moved around files to look a bit like this:
```
include/datasketches
count/
count_min.hpp
count_min_impl.hpp
common/
MurmurHash3.h
...
```

Thanks for your consideration.

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Examinez les dossiers de composants existants du repository et leurs répertoires include/, puis comparez-les avec l’organisation proposée de include/datasketches et le précédent pybind11 référencé dans l’issue. Avant de modifier des fichiers, confirmez que les maintainers se sont accordés sur une organisation cible ; la tâche serait terminée lorsqu’une organisation acceptée fournirait un unique chemin include aux systèmes de build externes.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
cpp
Domaine
build-system
Type d'issue
Refactorisation
Difficulté
5/5
Temps estimé
Plus d'une semaine
Activité
À l'abandon
Clarté
À clarifier
Accessibilité débutants
20/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.