Skip to content

Conversation

@namrata-ibm
Copy link

As Travis CI officially supports s390x builds, adding support for same.

@namrata-ibm namrata-ibm requested a review from alcaeus February 19, 2020 12:05
@jmikola
Copy link
Member

jmikola commented Feb 19, 2020

@namrata-ibm: Thanks for the contribution. I'm not sure this is necessary since we already have test coverage for zSeries on Evergreen, which is our internal CI. Our Evergreen matrix is much more comprehensive and we mainly rely on Travis for quicker PR feedback, so I think we'd do well to keep its matrix as small as possible (also given that it's a public resource). I'll defer to @alcaeus on this.

@alcaeus
Copy link
Member

alcaeus commented Feb 24, 2020

I agree with @jmikola here - since we're using public resources I'd rather keep the travis-ci pipeline small and use our own evergreen system to test additional platforms. Thanks for the suggestion!

@alcaeus alcaeus closed this Feb 24, 2020
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