apache / apache/datasketches-cpp
Reorganization proposal
- 主要言語
- C++
- スター
- 273
- フォーク
- 88
- 平均マージ
- 2日 2時間
- マージ済み PR(30日)
- 8
説明
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.
コントリビューションガイド
調査の方向性
リポジトリに既にあるコンポーネントフォルダーとその include/ ディレクトリを確認し、提案されている include/datasketches レイアウトおよび issue にリンクされた pybind11 の前例と比較してください。ファイルを変更する前に、maintainer が目標とする構成に合意していることを確認してください。完了とみなすには、外部のビルドシステムに単一の include パスを提供する、受け入れ済みのレイアウトが必要です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- cpp
- 領域
- build-system
- issue の種類
- リファクタリング
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 活発さ
- 停滞
- 明瞭さ
- 説明が足りない
- 初心者へのやさしさ
- 20/100