Build failed in Travis CI. (Works fine on desktop)

[jmschrack]

Build failed in Travis CI. (Works fine on desktop)

[gfscott]

Hm. It looks like maybe Travis doesn’t accept named capture groups in regex, so it’s failing on the ?<videoId> value.

I'll look into Travis (I haven't used it much) and consider some alternative approaches. Thanks for filing an issue!

[gfscott]

@jmschrack So I made a PR today to try to address this. I took the liberty of forking your repo and running it through Travis, and the build passed this time:

https://travis-ci.org/github/gfscott/jmschrack.github.io/jobs/662532397

If you're able, it would be great to confirm that this update is working for you before I merge it. If you could pull that branch and give it a try yourself, I'd be obliged. 🙏

As I suspected, the problem is the named capture groups, which were only added to Node with version 10. Since your build is set to use Node 8, those RegExes wouldn't work. Seems better to just use a more widely supported method, rather than requiring Node 10.