WIP: Remove 'regex-posix' dependency#289
WIP: Remove 'regex-posix' dependency#289tom-audm wants to merge 1 commit intosnapframework:masterfrom
Conversation
|
Reasons for removing
One solution is to, in addition to the work here, move |
|
As described in the docs, this change requires a user who truly does want to use regexes to write: import Text.Regex.Posix
assertBodyMatches "^Hello" (=~ "^Hello") rinstead of assertBodyContains "^Hello" r |
|
I'm broadly in favor of this kind of dependency pruning. The API breakage gives me pause. But it also doesn't seem like a super commonly used function. Pinging @gregorycollins. |
|
@tom-audm fwiw, the 3rd and 4th bullet points are currently being addressed... ;-) |
|
It will break test code, but I'm probably okay with it? We'll need a major version bump unfortunately |
|
Just chiming in here because I wanted to build more conveniently on windows. Have you considered replacing the |
|
I think the consensus is that we're just going to get rid of this dependency (and the function) altogether |
|
Yeah, I'm in favor of this change. |
WIP, please don't merge yet!