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

First attempt at CircleCI config #1087

Merged
merged 20 commits into from Feb 22, 2017

Conversation

tekknolagi
Copy link
Contributor

No description provided.

@tekknolagi
Copy link
Contributor Author

tekknolagi commented Feb 22, 2017

It wasn't supremely easy to translate Travis->CircleCI but I did it. Multiple versions coming next.

Tests currently take ~3min to run and start ~instantly.

@yunxing
Copy link
Contributor

yunxing commented Feb 22, 2017

This is awesome, CI immediately passes.

circle.yml Outdated
override:
- sudo add-apt-repository -y ppa:avsm/ppa
- sudo mkdir -p ~/aptcache/partial
- sudo touch ~/aptcache/lock
Copy link
Contributor

Choose a reason for hiding this comment

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

What's the lock for?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh I had to add this one time. Guess I could have done that via SSH. Can remove.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@yunxing I am slightly worried that this could be an issue in the future if the cache gets wiped. I might add them back in with a comment.

Copy link
Contributor

Choose a reason for hiding this comment

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

@tekknolagi ah, ok. Please feel free to add it back with a comment.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@tekknolagi tekknolagi merged commit d513c89 into reasonml:master Feb 22, 2017
sansthesis pushed a commit to sansthesis/reason that referenced this pull request Apr 19, 2017
Features:
* Cache apt packages
* Cache OCaml installs and packages
* Multi-version testing support (not enabled)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants