Reorganization proposal

未关闭
#419 6 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

评估

难度
5/5
预计耗时
一周以上
新手友好度
20/100
Issue 类型
重构
描述清晰度
需要澄清
活跃度
停滞
技术栈
cpp
领域
build-system

调研方向

检查 repository 现有的组件文件夹及其 include/ 目录，然后将它们与提议的 include/datasketches 布局以及 issue 中链接的 pybind11 先例进行比较。在更改文件之前，确认 maintainer 已就目标组织方式达成一致；只有存在一个能为外部构建系统提供单一 include 路径的已接受布局，才算完成。

由索引模型根据 Issue 内容生成。

描述

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.

主要语言
C++
星标
273
派生
88
平均合并
1 天 19 小时
30 天内合并 PR
9

贡献指南

打开贡献指南

从这里开始

  1. 先读完整个 Issue，再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库，在一个分支上完成修改。
  4. 提交 Pull Request，并在描述里引用这个 Issue 编号。

apache/datasketches-cpp 的其他 Issue

查看 apache/datasketches-cpp 的全部 Issue

相似的 Issue

更多 C++ Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。