apache / apache/datasketches-cpp

Reorganization proposal

Đang mở
#419 6 bình luận 0 reaction 0 người được giao Xem trên GitHub
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

Mở 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

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.