-
Notifications
You must be signed in to change notification settings - Fork 34
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
Handle lack of app/templates/head.hbs
more gracefully.
#7
Comments
@rwjblue (Sorry for the late reply.) Is the blueprint that creates one on install insufficient? Was it manually removed? Seems pretty straight forward to add in, just curious how it came up. |
@ronco I got this error when migrating to pods. I moved I know 'pods are dead' as we wait for the new module layout. But I thought using pods for the ui files was a good incremental step in my project.
|
So we don't hard error if a consuming app misplaced their head.hbs Resolves #7
Included a default template in v0.1.0 release. |
Currently, if you don't have the
app/templates/head.hbs
file present you get a very difficult to parse error like the following:It is pretty hard to track down unless you happen to know what ember-cli-head is doing internally. I think erroring hard is ok here, but it is also possible for us to default to an empty template (to prevent errors like this).
The text was updated successfully, but these errors were encountered: