Skip to content
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

Fix travis.yml in addon blueprint #7372

Merged
merged 1 commit into from
Oct 10, 2017

Conversation

simonihmig
Copy link
Contributor

After running ember-cli-update in an addon and pushing, the Travis build was broken with a yaml parsing error: https://travis-ci.org/kaliber5/ember-window-mock/builds/286204403

The changes were introduced in #7349, but this does indeed not seem like valid yaml (env being used as object as well as array). Travis docs show this: https://docs.travis-ci.com/user/environment-variables/#Global-Variables

After adding the additional matrix key, this fixed my build...

@@ -22,6 +22,7 @@ env:
global:
# See https://git.io/vdao3 for details.
- JOBS=1
matrix:
# we recommend new addons test the current and previous LTS
# as well as latest stable release (bonus points to beta/canary)
- EMBER_TRY_SCENARIO=ember-lts-2.8
Copy link
Member

@rwjblue rwjblue Oct 10, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this comment and the list items need to be indented an extra 2 spaces.

@rwjblue
Copy link
Member

rwjblue commented Oct 10, 2017

Thank you!

  • Can you create a new issue around adding tests for validating the emitted yml files?
  • Can you target this PR against release branch (and rebase) so that we can get this out in a point release?

@simonihmig simonihmig changed the base branch from master to release October 10, 2017 20:48
@rwjblue rwjblue merged commit 6f63322 into ember-cli:release Oct 10, 2017
@simonihmig simonihmig deleted the fix-addon-travis branch October 10, 2017 21:42
@Turbo87 Turbo87 added the bug label Oct 21, 2017
@Turbo87 Turbo87 changed the title [BUGFIX] Fix travis.yml in addon blueprint Fix travis.yml in addon blueprint Oct 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants