middleman/middlemanapp.com

Travis

tysongach opened this issue · 5 comments

tysongach commented

Our build has been breaking in Travis recently. It doesn’t appear to be tied to a specific commit being merged. It does appear to be tied to the external pipeline: https://travis-ci.org/middleman/middleman-guides#L585

I’ve tried upgrading the Node and JavaScript dependencies with no luck.

Also, there is a FIREBASE_TOKEN environment variable set in Travis. Is that used/needed? It seems to be throwing a warning:

The previous command failed, possibly due to a malformed secure environment variable.
yuya-matsushima commented

Fixed Travis' problem with updating Japanese document. #685
It seems that the node version is too old. (before 0.10.36, now 7.6)

The problem of FIREBASE_TOKEN still exists.

tysongach commented

Thanks for fixing, @yterajima!

Maybe @tdreyno knows about the FIREBASE_TOKEN environment variable.

tdreyno commented

FIREBASE_TOKEN can go, I think. It was for when we were hosted on Firebase's static hosting.

We should lock node to their stable 6.9.x

tysongach commented

@tdreyno 👍

I update Node to 7.7.1 in 1e008e6. I think Travis is looking at the .nvmrc file when it installs Node?

tysongach commented

I removed the FIREBASE_TOKEN environment variable, which removed the error being thrown. That, along with @yterajima’s change to make sure Node is installed, means Travis is all green now and the log is clean.

  • 👍1