apache / apache/datasketches-cpp
Reorganization proposal
- Ngôn ngữ chính
- C++
- Star
- 273
- Fork
- 88
- Merge trung bình
- 2 ngày 2 giờ
- Pull request đã merge (30 ngày)
- 8
Mô tả
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.
Hướng dẫn đóng góp
Hướng nghiên cứu
Xem xét các thư mục component hiện có của repository và các thư mục include/ của chúng, sau đó so sánh với layout include/datasketches được đề xuất và tiền lệ pybind11 được liên kết trong issue. Trước khi thay đổi các tệp, hãy xác nhận rằng các maintainer đã thống nhất về một tổ chức đích; công việc chỉ được xem là hoàn tất khi có một layout được chấp nhận, cung cấp một đường dẫn include duy nhất cho các hệ thống build bên ngoài.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- cpp
- Lĩnh vực
- build-system
- Loại issue
- Tái cấu trúc
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Cần làm rõ
- Mức phù hợp với người mới
- 20/100