Skip to content
This repository has been archived by the owner on Dec 31, 2022. It is now read-only.

chore(deps): upgrade Ember CLI to latest #216

Merged
merged 5 commits into from
Nov 16, 2020
Merged

Conversation

chriskrycho
Copy link
Member

@chriskrycho chriskrycho commented Nov 13, 2020

This is really preparatory work for other cleanup; it just gets us to the point where the test suite is actually useful again.

  • Update Ember CLI to current stable.
  • Because doing ☝️ breaks our test-version detection behavior and because we really, really don't need to support the old-style tests anymore, drop all of that functionality entirely and support only current versions of ember-qunit and ember-mocha.
  • Remove the component blueprints entirely, in favor of landing updated versions of them in improve component blueprint for octane #213.

We need to add support for testing against ember-qunit 5.0 and its blueprints in a future update, and we also need to go ahead and clean up the actual contents of these blueprints, but that's follow-on work!

@chriskrycho chriskrycho added the dependencies Pull requests that update a dependency file label Nov 13, 2020
@chriskrycho chriskrycho force-pushed the bump-ember-cli branch 2 times, most recently from f565101 to a697276 Compare November 14, 2020 20:25
BREAKING CHANGE: drops all support for pre-[RFC 0232][rfc] blueprints,
since these have long since been replaced.

Note that this commit also removes the component blueprints entirely, in
favor of landing updated versions of them in [another PR][pr].

[rfc]: https://emberjs.github.io/rfcs/0232-simplify-qunit-testing-api.html
[pr]: #213
@chriskrycho
Copy link
Member Author

chriskrycho commented Nov 14, 2020

Travis is borked. AppVeyor and Azure Pipelines are both testing Windows. 🤦‍♂️ #215 looks, uhh, even more worthwhile now.

Edit: Travis was borked b/c I hadn't updated the lock file to match package.json after making a fix. Fixed. But also: #217 == 🎉

@chriskrycho
Copy link
Member Author

Note that the CI changes I made here get everything to pass, but will be superseded by #217. It's probably easiest (no rebase to manage) if we land this first, then land that as it should be a clean diff.

Copy link
Member

@dfreeman dfreeman left a comment

Choose a reason for hiding this comment

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

Spot-checking this, everything looks good to me. Since CI's happy, I'm happy :shipit:

@chriskrycho chriskrycho merged commit eec179d into master Nov 16, 2020
@chriskrycho chriskrycho deleted the bump-ember-cli branch November 16, 2020 16:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants