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

Use configstore for storing versionCheck information, and only store the timestamp when the version is checked from npm #1323

Merged

Conversation

andycrum
Copy link
Contributor

@andycrum andycrum commented Jul 9, 2014

Per #1223, using configstore instead of yam for storing lastVersionCheck and newestVersion for the UpdateChecker.

@stefanpenner
Copy link
Contributor

👍

  • changelog entry
  • test (if you have time)

…the timestamp when the version is checked from npm

Add CHANGELOG entry and tests for UpdateChecker/make UpdateChecker more easily mockable
@andycrum
Copy link
Contributor Author

andycrum commented Jul 9, 2014

@stefanpenner added changelog entry and tests. Had to move some UpdateChecker functionality around to make it more easily testable, but should be working fine.

stefanpenner added a commit that referenced this pull request Jul 9, 2014
…orage

Use configstore for storing versionCheck information, and only store the timestamp when the version is checked from npm
@stefanpenner stefanpenner merged commit 54eeb47 into ember-cli:master Jul 9, 2014
@stefanpenner
Copy link
Contributor

thanks!

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