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

Remove .travis.yml #2876

Merged
merged 1 commit into from
Sep 27, 2018
Merged

Remove .travis.yml #2876

merged 1 commit into from
Sep 27, 2018

Conversation

jacobherrington
Copy link
Contributor

We decided to not require Ruby 2.3, so we should
make sure we are aware of any issues on Ruby 2.2
and Ruby 2.5.1 once broke Solidus, so it seems smart
to test against it as well.

Creating this PR to test CI and facilitate discussion.

Related:
#2794
#2825

@jacobherrington jacobherrington changed the title Test against Ruby 2.2.10 and 2.51 Test against Ruby 2.2.10 and 2.5.1 Sep 27, 2018
Copy link
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

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

We don't build on Travis. We build on CircleCI

Also, I do not think it makes sense to test all supported Ruby versions.

@jacobherrington
Copy link
Contributor Author

image 🤦‍♂️

@jacobherrington
Copy link
Contributor Author

@tvdeyen in that case can we remove .travis.yml? Would help blind people like me not to do this.

@tvdeyen
Copy link
Member

tvdeyen commented Sep 27, 2018 via email

@jacobherrington
Copy link
Contributor Author

Added a commit, should I make a new PR or use this one?

@jacobherrington jacobherrington changed the title Test against Ruby 2.2.10 and 2.5.1 Remove .travis.yml Sep 27, 2018
Copy link
Contributor

@ericsaupe ericsaupe 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 squash the commits into just the one for removing .travis.yml. Good clean up though!

We aren't testing with TravisCI, so this file is just
confusing.
@jacobherrington
Copy link
Contributor Author

@ericsaupe good call. done!

Copy link
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

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

Thanks

@kennyadsl kennyadsl merged commit 07a8b7e into solidusio:master Sep 27, 2018
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.

4 participants