-
-
Notifications
You must be signed in to change notification settings - Fork 67
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
warning introduced by plugins not caching, is not as actionable as it could #92
Comments
Also, we need docs here explaining how to fix this. |
FWIW, the AST plugin system is not public API. I didn't document this for that reason. |
It's documented on the home page. (.___________ .) I'm currently using this in my addon https://github.com/tim-evans/ember-page-title, and am encountering this error. |
@tim-evans - What home page? |
Oh, the README. I figured what I was doing wrong :) |
Sorry if I'm being a bother~ |
I'm not sure what release/update exactly introduced it, but I'm not getting these warnings on my project:
There's two different modules complaining about this issue, caused by two other modules. I didn't find any open issues for the other modules... Does anybody have a hint how to fix this? That could be a starting point for proper docs. |
I think it's using the wrong hook in See ember-cli/ember-page-title@8918d70 for what I did to fix it. |
Thanks for the link, but as a user of those addons (not an author), that doesn't help me. I've got no idea what's going on in that code... |
@jzaefferer please open an issue on the offending add-on. |
Filed on for ember-css-modules: salsify/ember-css-modules#50 |
https://github.com/ember-cli/ember-cli-htmlbars/pull/90/files#diff-fc695117845e9b76dfb1841d2221ac7cR133
should include a URL to "how to fix it".
The text was updated successfully, but these errors were encountered: