Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

enable travis #22

Merged
merged 25 commits into from
Dec 6, 2017
Merged

enable travis #22

merged 25 commits into from
Dec 6, 2017

Conversation

NobbZ
Copy link
Member

@NobbZ NobbZ commented Jun 10, 2017

This PR aims to enable travis for the track and has the following subgoals:

  • install a local trackler and x-api
  • install a local client
  • check if config is valid
  • fetch every single exercise from local X-API
  • solve every single exercise by copying example-file over stub-file and run the tests
  • collect status of tests and have a proper exit code at the end

FIXES #3

@yurrriq
Copy link
Member

yurrriq commented Jun 10, 2017

Any interest in using Nix (language: nix and nix-build)?

@NobbZ
Copy link
Member Author

NobbZ commented Jun 10, 2017 via email

@yurrriq
Copy link
Member

yurrriq commented Jun 10, 2017

It would probably slow it down tbh. I'm just a Nix fanboy, basically. I'm totally cool with a Stack-based solution. Thanks for doing this!

@NobbZ
Copy link
Member Author

NobbZ commented Jun 10, 2017

Hehe, fanboy-ism ;)

Problem there is, that we will hit a massive wall with everything that eats our precious time. With a cold cache there is already half an hour taken away by compiling idris, also we know that idris isn't very fast when it comes to compiling idris programs.

I really wan't to avoid failing builds because of hitting the 90 minute limit.

@yurrriq
Copy link
Member

yurrriq commented Jun 10, 2017

Ah, so if we're a little clever, we can hit a Hydra binary cache with Nix too. FWIW the Idris infrastructure is a little broken atm, though that would affect us here.

@NobbZ
Copy link
Member Author

NobbZ commented Jun 10, 2017

Okay, its done!

Collecting the exit-code does work, but currently it only shows us when compilation failed, not failed tests. Since the testsuite still exits with 0 on fail.

So to have this work properly we will need to tinker around with the test suite, which I will tackle in a separate PR. Perhaps I will simply switch to idris-testing while doing this.

@NobbZ NobbZ changed the title WIP: enable travis enable travis Jun 10, 2017
@yurrriq
Copy link
Member

yurrriq commented Jun 10, 2017

Nice! Yeah, you can use idris-testing's Reporting namespace (can't remember which module just now) and check for any not or similar and use the exit functions from System.

This was referenced Jun 10, 2017
@NobbZ NobbZ mentioned this pull request Jun 11, 2017
@yurrriq yurrriq merged commit 87ecf62 into exercism:master Dec 6, 2017
@yurrriq
Copy link
Member

yurrriq commented Dec 6, 2017

Thanks. I've never resolved conflicts in the GitHub UI before. I won't be doing that again :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants