Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove findSubstring and findSubstrings #181

Merged
merged 2 commits into from
Aug 23, 2020

Conversation

bgamari
Copy link
Contributor

@bgamari bgamari commented Jul 13, 2019

These were deprecated in 2008 (a65029c)
in favor of breakSubstring and findSubstrings appears to be incorrect
(see #180).

These were deprecated in 2008 (a65029c)
in favor of breakSubstring and findSubstrings appears to be incorrect
(see haskell#180).
@bgamari
Copy link
Contributor Author

bgamari commented Jul 13, 2019

I recommend that we release #179 as-is for 8.8.1 to avoid a major version bump which would delay the release even further. We can then release this in a subsequent major bump.

@hvr
Copy link
Member

hvr commented Dec 18, 2019

@bgamari I'm very reluctant to perform a major version bump (especially for a library such as bytestring which is depended upon by everyone) for the mere purpose of removing deprecated functions which pose no harm whatsoever

or put differently, I'd rather collect all low-priority API removals unti the point where something else forces a major version bump, and then execute the deprecation-removals at once.

@hvr hvr added the blocked: next-major-version-bump blocked until a major version bump is happening label Dec 18, 2019
@sjakobi sjakobi mentioned this pull request Dec 20, 2019
1 task
@vdukhovni
Copy link
Contributor

I fixed the findSubstrings tests in #204 . It is was merely an issue with 8-bit truncation of unicode strings introducing more matching substrings.

@vdukhovni
Copy link
Contributor

I fixed the findSubstrings tests in #204 . It is was merely an issue with 8-bit truncation of unicode strings introducing more matching substrings.

With #204 merged, this PR is no longer a blocker for resolving CI issues. It might still be appropriate to merge it at some future date, but is now less pressing.
@cartazio

@sjakobi sjakobi linked an issue Jun 25, 2020 that may be closed by this pull request
@sjakobi sjakobi mentioned this pull request Aug 19, 2020
Copy link
Contributor

@Bodigrim Bodigrim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is good to go as a part of 0.11.0.0 release.

@Bodigrim Bodigrim added this to the 0.11.0.0 milestone Aug 21, 2020
@Bodigrim Bodigrim merged commit 829771b into haskell:master Aug 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked: next-major-version-bump blocked until a major version bump is happening
Projects
None yet
Development

Successfully merging this pull request may close these issues.

findSubstrings is broken
4 participants