shexSpec / shex

ShEx language issues, including new features for e.g. ShEx2.1
24 stars 8 forks source link

Make keyword 'a' case insensitive. #101

Closed thewillyhuman closed 4 years ago

thewillyhuman commented 4 years ago

Hi, I'm currently implementing shex-lite (early-early version yet), which is an small subset of ShEx. I know that the keyword 'a' comes from m3 and later from turtle, but maybe, as the language shex is completly case insensible you might consider to make the 'a' keyword case insensitive too. It will simplify the Antlr grammar by just declaring case insentivity at parsing time, instead of defining a fragment for every character in upper and lower case.

By the way any hint to define shex-lite subset will be helpful.

Thanks.

ericprud commented 4 years ago

Oof, you're making me wrestle with a sunk-cost fallacy.

I think we should be compatible with how Turtle and SPARQL "ought to be", so if we decide to make it case-insensitive, we're signalling that it ought to be case-insensitive in the rest of the stack.

alejgh commented 4 years ago

Thank you for the response @ericprud. We imagined that it would not be correct going against Turtle and SPARQL case-insensitivity rules, but we still wanted to receive your input on the subject.

We'll keep the 'a' keyword case-sensitive in shex-lite too 😊. This issue can be closed.