I was looking for a way to lint handlebars files in discourse. I did find a command that seems to run but doesn’t make any changes or list issues.
yarn ember-template-lint --fix path/to/files
I’m not sure if its even supposed to work at this point.
yarn ember-template-lint <dir> should be the correct command.
8 Likes
Thanks. But doesn’t seem to report anything back. I tried to run it on a broken file but it doesn’t display errors or fix them.
1 Like
@cvx you just implemented this on our plugin workflows – is there something special we need to do to use this?
3 Likes
No magic required 
but make sure you run the command in the root directory of Discourse. This got me a couple times already.
Compare:
# in [discourse]/app/assets/javascripts
$ yarn ember-template-lint .
✨ Done in 2.57s.
# in [discourse]
$ yarn ember-template-lint app/assets/javascripts
app/assets/javascripts/discourse/app/templates/tags.hbs
4:0 error Unexpected {{debugger}} usage. no-debugger
4:0 error Usage of triple curly brackets is unsafe no-triple-curlies
3:15 error Incorrect indentation for `d-section` beginning at L1:C0. Expected `{{/d-section}}` ending at L3:C15 to be at an indentation of 0 but was found at 1. block-indentation
2:4 error Incorrect indentation for `{{outlet}}` beginning at L2:C4. Expected `{{outlet}}` to be at an indentation of 2 but was found at 4. block-indentation
1:23 error you must use double quotes in templates quotes
✖ 5 problems (5 errors, 0 warnings)
error Command failed with exit code 1.
Also, keep in mind it will use our config (.template-lintrc.js) so it might not report all expected issues as some rules are currently disabled.
@justin I just realized that our plugin GitHub Actions workflows currently don’t share this config. We should change that at some point. 
9 Likes
Will this run in plugins/? I’m betting that’s where @fzngagan is interested in using this.
5 Likes
If the plugin is located in plugins directory, sure!
$ yarn ember-template-lint plugins
plugins/poll/assets/javascripts/discourse/templates/modal/poll-breakdown.hbs
1:22 error you must use double quotes in templates quotes
4 Likes
I’m pretty sure I’m doing the same thing, but will try again and let you know.
Ember template lint cant fix everything, just like eslint, and actually ember template lint can only fix a few things, unlike eslint.
It will mostly report errors, but you manually have to fix them.
3 Likes
Here’s what I’ve tried.
I tried by breaking a rule with discourse itself. Works
I tried by breaking a rule with discourse-custom-wizard plugin which was in the discourse directory as a symlink. Doesn’t work
I tried the same with the poll plugin which is the part of the discourse repo. I also tried it with other plugins which are shipped with discourse. Works
I tried by breaking a rule with discourse-custom-wizard plugin again, this time copying it to the discourse’s plugin folder. Doesn’t work
I tried by breaking a rule with discourse-assign plugin placing the folder in plugins. Doesn’t work
1 Like
For me:
works
yarn ember-template-lint plugins/discourse-pfaffmanager/assets/javascripts/discourse/templates/user/servers/show.hbs
does not work
yarn ember-template-lint plugins/discourse-pfaffmanager/assets/javascripts/discourse/templates/user/servers/
And should my assets be inside of app?
1 Like
Ah, yes. The issue is that ember-template-lint automatically takes .gitignore into account. And non-bundled plugins are ignored.
So, to run template linting on all plugins:
yarn ember-template-lint --no-ignore-pattern plugins
# or any other directory, e.g. a single plugin:
yarn ember-template-lint --no-ignore-pattern plugins/discourse-calendar
5 Likes
Thanks. Works like a charm.
3 Likes
Thanks for that! Also, is there a way to get it to auto fix stuff? I tried adding --fix to no avail.
1 Like
It works, but:
- there are only a handful of rules that are auto-fixable (see: https://github.com/ember-template-lint/ember-template-lint#rules)
- of those only two are in the “recommended” ruleset (link-rel-noopener and require-button-type)
3 Likes