search

starteam / starcellbio_html

2 stars 7 forks source link

Added settings for travis to use old build instance #784

Closed blarghmatey closed 6 years ago

blarghmatey commented 6 years ago

The updated build instance is causing tests to fail. Attempting to run on the previous host version to resolve failures.

What are the relevant tickets?

What's this PR do?

Fixes travis builds

Where should the reviewer start?

Travis

How should this be manually tested?

Any background context you want to provide?

The latest image in Travis is causing builds to fail, most likely due to one of the updates of Python or Node. This PR pins the image version to the previous release.

Screenshots (if appropriate)

What GIF best describes this PR or how it makes you feel?

jmclaus commented 6 years ago

@blarghmatey Thanks. :+1: Since we are meddling with .travis.yml, shouldn't we fix errors reported by Travis WebLint on that same file? (perhaps in another PR)

For example, it mentions your repository must be feature flagged for the group setting to be used. (not sure it's relevant in our case)

blarghmatey commented 6 years ago

@jmclaus feel free to modify the travis file as necessary.

jmclaus commented 6 years ago

@blarghmatey Thanks for the fix.