Skip to content
This repository was archived by the owner on Sep 20, 2023. It is now read-only.

[WIP] Fixed #98, add Coveralls integration #99

Merged
merged 2 commits into from
Sep 3, 2016

Conversation

adinapoli
Copy link
Contributor

I copied verbatim the options from my other repo, so that should do the trick, but I think we
will need to explicitly enable Foundation in Coveralls.

I think this requires one of the owner of the foundation repo (probably Vincent) to sign up and add the project, as I (as a member) can only enable my own fork, not the real deal:

screen shot 2016-09-03 at 09 39 46

@vincenthz vincenthz added the C - tests tests stuff label Sep 3, 2016
@NicolasDP
Copy link
Member

I have added it to coverall, is it still WIP ? https://coveralls.io/github/haskell-foundation/foundation

@adinapoli
Copy link
Contributor Author

@NicolasDP Usually once Coveralls is fully set we should get it to post on each new open PR and get a new check, as below:

screen shot 2016-09-03 at 16 52 59

screen shot 2016-09-03 at 16 53 13

I suspect though, that since we added the permissions at a later stage we won't see those things now. Probably it's just easier to merge this and add small incremental tweaks until we get it to work 😉

@NicolasDP NicolasDP merged commit f8f801b into haskell-foundation:master Sep 3, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
C - tests tests stuff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants