Skip to content

WIP: Use cabal-doctest - #245

Closed
phadej wants to merge 4 commits into
haskell:masterfrom
phadej:cabal-doctest
Closed

phadej wants to merge 4 commits into
haskell:masterfrom
phadej:cabal-doctest

Conversation

@phadej

@phadej phadej commented May 12, 2017

Copy link
Copy Markdown

Fixes #196

@eborden

eborden commented May 15, 2017

Copy link
Copy Markdown
Collaborator

Looks like this is failing across the board.

setup: Error: Could not find test program "dist/build/doctests/doctests"

Comment thread tests/doctests.hs
, "-XDeriveDataTypeable"
, "-package-db dist/package.conf.inplace"
, "-package network"
, "Network"

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

previously we tested only Network module.

@phadej

phadej commented May 18, 2017

Copy link
Copy Markdown
Author

not sure what happens on travis; maybe old cache haunts there. Would investigate if maintainers are willing to pursuit this approach.

@eborden

eborden commented May 31, 2017

Copy link
Copy Markdown
Collaborator

I'm in favour of this approach. There certainly seems to be some non determinism on travis.

Same source, separate results:

@hvr hvr mentioned this pull request Jun 1, 2017
@phadej phadej mentioned this pull request Dec 10, 2017
@kazu-yamamoto kazu-yamamoto mentioned this pull request Dec 15, 2017
1 of 4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants