Topic/travis - Githubissues

yav / haskell-lexer

A fully compliant Haskell 98 lexer.

MIT License

15 stars 7 forks source link

Topic/travis #4

Closed tmcgilchrist closed 3 years ago

tmcgilchrist commented 7 years ago

Adds a travis ci build config.

Also pulls in some other perhaps questionable changes like:

GHC supported ranges, I went with 7.8, 7.10, 8.0 and head (easy to change for more if further backwards compatibility is required)
version range on base (cabal check encouraged having such a range)
silencing warnings and some CPP
removing -02and bumping cabal version range, again cabal check encouraged this.

Dropping the running of cabal check might remove the need to do some of these things.

yav commented 7 years ago

Thanks for doing this. Could you modify the pull request to keep the -O2 flag? I know cabal check complains about it, but I run the code more often then I compile it, so I'd rather have the flag in there, until we run some benchmarks and determine that it is not needed.

tmcgilchrist commented 7 years ago

Sure no problem.

tmcgilchrist commented 7 years ago

To get the build status to turn up on a PR you'll need to enable it in travis website.

tmcgilchrist commented 7 years ago

@yav Still interested in merging this?

tmcgilchrist commented 3 years ago

There are now better options for CI, namely Github Actions.