Reorganization proposal

Đang mở
#419 6 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
Mức phù hợp với người mới
20/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Cần làm rõ
Mức độ hoạt động
Đình trệ
Công nghệ
cpp
Lĩnh vực
build-system

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.

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 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.

Ngôn ngữ chính
C++
Star
273
Fork
88
Merge trung bình
1 ngày 19 giờ
Pull request đã merge (30 ngày)
9

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của apache/datasketches-cpp

Tất cả issue của apache/datasketches-cpp

Issue tương tự

Thêm issue về C++

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.