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

Improve travis execution time #21

Merged
merged 6 commits into from
Oct 16, 2017
Merged

Conversation

simondobson
Copy link
Contributor

Closes #16 and #17

@simondobson simondobson requested a review from javigomez October 3, 2017 08:10
@simondobson simondobson force-pushed the improve-travis-execution-time branch from 78299be to 09ffb81 Compare October 3, 2017 08:30
.travis.yml Outdated
bundler: true
directories:
- /home/travis/.rvm/

Choose a reason for hiding this comment

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

mmm, doesn't seem to work for Ruby installation

Before caching:
beforecaching

After caching:
aftercaching


cache:
bundler: true

Choose a reason for hiding this comment

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

This one does the trick:

Before:
screen shot 2017-10-04 at 18 27 36

After:

screen shot 2017-10-04 at 18 27 16

Copy link

@javigomez javigomez left a comment

Choose a reason for hiding this comment

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

I would suggest to remove the rvm caching, doesn-t seem to work. The bundler cache seems good.

@simondobson simondobson force-pushed the improve-travis-execution-time branch 6 times, most recently from c1ab9fa to 013fe16 Compare October 4, 2017 17:42
@simondobson simondobson force-pushed the improve-travis-execution-time branch from 013fe16 to 59c5a32 Compare October 4, 2017 17:45
@javigomez
Copy link

I think travis fails because you haven't added rubocop to the Gemfile https://github.com/Typeform/create-api-gem/blob/master/Gemfile

@simondobson
Copy link
Contributor Author

Really weird, it's still not working :(

.travis.yml Outdated
bundler: true

script: rubocop && rake test

Choose a reason for hiding this comment

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

try this:

script:
  bundle exec rubocop
  bundle exec rake test

@simondobson
Copy link
Contributor Author

Yes! Working again :) - thanks @javigomez

@simondobson simondobson merged commit 9ff7af7 into master Oct 16, 2017
@simondobson simondobson deleted the improve-travis-execution-time branch October 16, 2017 13:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants