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

Make CI faster especially on PR builds #287

Merged
merged 1 commit into from
May 20, 2019
Merged

Conversation

KtorZ
Copy link
Member

@KtorZ KtorZ commented May 20, 2019

Issue Number

#242

Overview

- Made PR builds operating slightly differently than the "standard build".
  The idea is to only do what we really need to do for PRs and keep the
  more expensive operations on merges or cron jobs.

- Prune a bit some of the `.stack` internals in order to reduce the size
  of the cached directory. This should speed up a bit the cache
  downloads and uploads of each job.

- Remove the 'rust' language to avoid having to setup rust on each job.
  In the end, we only need rust in order to install rust binaries, after
  what, we only need the statically linked binaries.

- Split the compilation step into three jobs stages:
  - rust stuff
  - stackage snapshot
  - haskell project (only source and bench, tests are compiled in their
    corresponding test jobs)

Comments

- Made PR builds operating slightly differently than the "standard build".
  The idea is to only do what we really need to do for PRs and keep the
  more expensive operations on merges or cron jobs.

- Prune a bit some of the `.stack` internals in order to reduce the size
  of the cached directory. This should speed up a bit the cache
  downloads and uploads of each job.

- Remove the 'rust' language to avoid having to setup rust on each job.
  In the end, we only need rust in order to install rust binaries, after
  what, we only need the statically linked binaries.

- Split the compilation step into three jobs stages:
  - rust stuff
  - stackage snapshot
  - haskell project (only source and bench, tests are compiled in their
    corresponding test jobs)
@KtorZ KtorZ self-assigned this May 20, 2019
@KtorZ KtorZ requested a review from Anviking May 20, 2019 15:55
@KtorZ
Copy link
Member Author

KtorZ commented May 20, 2019

Plan is to bypass CI here since this is actually updating the whole CI and fixing quite a bunch of issues we have these days.

@KtorZ KtorZ merged commit 24d0265 into master May 20, 2019
@iohk-bors iohk-bors bot deleted the KtorZ/review-CI-caching branch May 20, 2019 16:04
@KtorZ KtorZ mentioned this pull request May 21, 2019
1 task
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.

None yet

2 participants