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

Travis CI enabled #55

Merged
merged 6 commits into from
Nov 12, 2018
Merged

Travis CI enabled #55

merged 6 commits into from
Nov 12, 2018

Conversation

johnsyweb
Copy link
Contributor

Context

Since we released to RubyGems, a few little niggles have appeared.

Change

  • Bump Ruby, RubyGems & Bundler in Travis CI config
  • Update build badge to Travis CI
  • Update change log

Considerations

Will release as 0.3.1

Cc

@orien, @joesustaric

@@ -1,5 +1,8 @@
sudo: false
language: ruby
rvm:
- 2.4.2
before_install: gem install bundler -v 1.16.1
- 2.4.5
Copy link
Contributor

Choose a reason for hiding this comment

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

no 2.3 or 2.2 ? I think that if this project is trying to promote better gem lib freshness.. I would suspect those projects might be not so fresh with their ruby versions as well.. just a thought..

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Attempted in a9487b1. Fixed in 6c27f57.

.travis.yml Outdated
- 2.5.3
before_install:
- gem update --system
- gem install bundler -v 1.17.1
Copy link
Contributor

Choose a reason for hiding this comment

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

Why do we need to specify the version of bundler?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We don't. Addressed in 0e090f3.

@@ -16,7 +16,7 @@ to get regular dependency updates into production.
- Vulnerability advisory information using [bundler-audit](https://github.com/rubysec/bundler-audit)
- Links to the home page, source code and change log (where available) of each gem

## Development status [![Build status](https://badge.buildkite.com/d7db34f910131ff2a03d31dcc0ee960a3bc5f0df2c42ec4eb4.svg?branch=master&style=flat-square)](https://buildkite.com/envato-marketplaces/unwrappr)
## Development status [![Build Status](https://travis-ci.org/envato/unwrappr.svg?branch=master)](https://travis-ci.org/envato/unwrappr)
Copy link
Member

Choose a reason for hiding this comment

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

I'd be nice if they had a flat style like the other badges in the readme.

@johnsyweb johnsyweb merged commit 007b22e into master Nov 12, 2018
@johnsyweb johnsyweb deleted the paj/travis-bump-rubygems branch November 12, 2018 10:43
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.

3 participants