Skip to content

Update build matrix to cover TS + Ember CLI versions #211

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

Merged
merged 4 commits into from
Jun 4, 2018
Merged

Conversation

dfreeman
Copy link
Member

@dfreeman dfreeman commented May 15, 2018

Ok, #210 is merged and the build is green, so it's probably time for me to write up a proper description here.

This updates our testing approach so that rather than hitting a bunch of different versions of Ember, we run both the browser and the node tests against:

  • the current dependencies in our lockfile
  • the current ember-cli release
  • the current ember-cli beta
  • the current typescript release
  • the current typescript beta

Appveyor continues to only run against our default dependencies as before.

A few things we could opt to do down the line:

  • expand our coverage to include older versions of ember-cli and/or typescript, once we firm up our official support policy
  • adopt Travis stages to e.g. gate the versioned jobs on the defaults one so that we don't waste cycles on runs that will definitely fail

@dfreeman dfreeman changed the base branch from update-project to master May 15, 2018 02:43
@dfreeman dfreeman mentioned this pull request May 21, 2018
Copy link
Member

@chriskrycho chriskrycho left a comment

Choose a reason for hiding this comment

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

Looks good, let’s do it!

@dfreeman dfreeman merged commit 6c9371c into master Jun 4, 2018
@dfreeman dfreeman deleted the build-matrix branch June 4, 2018 14:44
dfreeman added a commit that referenced this pull request Aug 21, 2018
Update build matrix to cover TS + Ember CLI versions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants