grab probably shouldn't be case sensitive
Closed this issue · 5 comments
tomjaguarpaw commented
grab
is case sensitive and probably shouldn't be
https://github.com/ndmitchell/neil/blob/master/src/Cabal.hs#L225
ndmitchell commented
The fact cabal isn't case sensitive is a massive misfeature. Think of a good language. It should have a canonical form. Given it uses kebab casing, lower case is the only thing that makes sense. Part of the idea of grab is to enforce that.
tomjaguarpaw commented
That's reasonable.
ndmitchell commented
Do I not have a check somewhere that all cabal fields are lowercase? If I don't I should. If I do, perhaps it wants moving before the first grab.
tomjaguarpaw commented
That check is "Incorrect declaration style" and happens at https://github.com/ndmitchell/neil/blob/master/src/Cabal.hs#L245, after grab
.
ndmitchell commented
Fixed as part of #27.