haskell-streaming / streaming-bytestring

effectful sequences of bytes; an alternative no-lazy-io implementation of Data.ByteString.Lazy
BSD 3-Clause "New" or "Revised" License
16 stars 11 forks source link

multiple copies of code in source tree #34

Closed chessai closed 3 years ago

chessai commented 3 years ago

looks like we have

❯ tree lib
lib
├── Data
│   └── ByteString
│       ├── Streaming
│       │   ├── Char8.hs
│       │   └── Internal.hs
│       └── Streaming.hs
└── Streaming
    ├── ByteString
    │   ├── Char8.hs
    │   └── Internal.hs
    └── ByteString.hs

5 directories, 6 files

Where each module in Data/ByteString/ and Streaming/ are equivalent. What is the reason for the duplication?

chessai commented 3 years ago

Is the idea to deprecate the old Data.ByteString.* modules in favour of the new ones, while keeping both around?

chessai commented 3 years ago

Ah, I forgot about this. Yes, that is the point (as indicated by the deprecation pragmas). Deprecations require a majour version bump - so we would need to bump to 0.2.

fosskers commented 3 years ago

Ah, I didn't bump it on purpose because there were technically no breaking changes. I was going off of SemVer logic - is PVP different on that particular issue?