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

No longer check dependencies #267

Merged
merged 1 commit into from
Nov 14, 2015
Merged

No longer check dependencies #267

merged 1 commit into from
Nov 14, 2015

Conversation

seanpdoyle
Copy link
Contributor

Closes #262.

Managing user dependencies within app code is more trouble than it's
worth. Being explicit about our EmberCLI support in our README.md and
keeping the addon version in lock-step with the gem will be a better
solution long term.

Since EmberCLI now writes to STDERR, which EmberCLI-Rails
pipes to a file and monitors, front-end failures will
communicated to users through normal Error-raising means.

@seanpdoyle seanpdoyle force-pushed the sd-dependencies branch 3 times, most recently from f31fee6 to 3d2af74 Compare November 14, 2015 02:36
Closes [#262].

Managing user dependencies within app code is more trouble than it's
worth. Being explicit about our EmberCLI support in our `README.md` and
keeping the addon version in lock-step with the gem will be a better
solution long term.

Since EmberCLI now [writes to STDERR][stderr], which EmberCLI-Rails
[pipes to a file and monitors][error-file], front-end failures will
communicated to users through normal Error-raising means.

[#262]: #262
[stderr]: ember-cli/ember-cli#5039
[error-file]: #245
@seanpdoyle seanpdoyle merged commit b8a8452 into master Nov 14, 2015
@seanpdoyle seanpdoyle deleted the sd-dependencies branch November 14, 2015 02:43
seanpdoyle added a commit that referenced this pull request Nov 14, 2015
As of [#267], we no longer check the versions of our dependencies,
therefor `Helpers.match_version?` is unused and no longer necessary.

[#267]: #267
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.

1 participant