apache / apache/datasketches-cpp

Reorganization proposal

オープン
#419 コメント 6 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
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

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。