apache / apache/datasketches-cpp
Reorganization proposal
- Vorherrschende Sprache
- C++
- Sterne
- 273
- Forks
- 88
- Ø Merge
- 2 T. 2 Std.
- Gemergte PRs (30 T.)
- 8
Beschreibung
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.
Beitragsleitfaden
Rechercherichtung
Überprüfe die vorhandenen Component-Ordner des Repositorys und deren include/-Verzeichnisse und vergleiche sie anschließend mit dem vorgeschlagenen include/datasketches-Layout sowie dem in der Issue verlinkten pybind11-Vorbild. Bevor du Dateien änderst, bestätige, dass sich die Maintainer auf eine Zielorganisation geeinigt haben; abgeschlossen wäre die Aufgabe erst, wenn ein akzeptiertes Layout vorhanden ist, das Build-Systemen außerhalb des Repositorys einen einzigen include-Pfad bereitstellt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- cpp
- Bereich
- build-system
- Issue-Typ
- Refactoring
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 20/100