haskell / haskell/attoparsec

IsString should not be implemented

Open
#124 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Haskell
Stars
531
Forks
98
PR merge metrics
No merged PRs in 30d

Description

I am a bit torn by the `IsString` instance. On the one hand, it makes it easy to parse literal text. On the other hand, in parsers I often don't really want to be parsing just text when I put things in quotes in BNF. Usually something like "foo" really means parse the token "foo", and also deal with trailing whitespace. I want to make this magic happen by providing my own version of `IsString`, but since attoparsec has it's own, I'm stuck. I suggest the library does not provide this instance or that this is made overlappable, since Haskell gives me no mechanism for hiding it or selecting my own. Currently I'm dealing with a duplicate instance declaration rather than an overlap, and I can't force the compiler to select my version.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.