search

evincarofautumn / kitten

A statically typed concatenative systems programming language.
http://kittenlang.org/
Other
1.09k stars 39 forks source link

Add hlint to test deps #48

Closed perimosocordiae closed 11 years ago

perimosocordiae commented 11 years ago

Sorry for the commit-spam; it's just a one-line change.

perimosocordiae commented 11 years ago

Bump.

evincarofautumn commented 11 years ago

Oh right, I got sidetracked on types and didn’t take care of this. Sorry!

I don’t really like adding HLint as a Cabal dependency because it installs a local version when you use cabal-dev. Not really sure how to fix that...maybe we should add cabal install hlint to make deps instead?

strager commented 11 years ago

Why not lint if hlint is available, and print a warning otherwise?

On Fri, Jul 5, 2013 at 2:11 PM, Jon Purdy notifications@github.com wrote:

Oh right, I got sidetracked on types and didn’t take care of this. Sorry!

I don’t really like adding HLint as a Cabal dependency because it installs a local version when you use cabal-dev. Not really sure how to fix that...maybe we should add cabal install hlint to make deps instead?

— Reply to this email directly or view it on GitHubhttps://github.com/evincarofautumn/kitten/pull/48#issuecomment-20534574 .

perimosocordiae commented 11 years ago

I like the warning idea. Even simpler.