-
Notifications
You must be signed in to change notification settings - Fork 28
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
Experiment: attempt to restore Travis CI #93
Experiment: attempt to restore Travis CI #93
Conversation
… by removing over-escaping of before_script commands
Aha: the problem may was within one of the nested build output steps:
So updating and/or tuning a few of the |
…allation by removing over-escaping of before_script commands" This reverts commit 6be2bdd.
…ll via 'yarn install'
NB: There are other dependency updates that could be beneficial to apply; the goal in this pull request is to get continuous integration working again first, and then dependency freshness can be applied as a subsequent set of changes. |
Nope, it's been a long day. The
This has been resolved in another codebase by upgrading the Yarn version registreerocks/registree-core#370 . In the builds for this repository we are dependent on the underlying OS environment. Can/should we upgrade the Yarn version within the CI container, either by selecting a more up-to-date environment, or by performing a runtime upgrade? |
… reinstall via 'yarn install'" This reverts commit 137993f.
cc @larixer |
Looks good to me, thank you! |
What's the problem this PR addresses?
Continuous integration via Travis seems broken at the moment; see this example build log from pull request #92.
The problem is potentially due to over-escaping of the
before_script
commands, based on the observed output:Before opening and/or merging further changes it'd be good to get tests green again.
How did you fix it?
This isn't yet a confirmed fix; but the approach taken is to remove the outer double-quotes from the command specification.