# הצעה: הפעלת כלל \`eol-last\` עבור eslint ו-ember-template-lint

**URL:** https://meta.discourse.org/t/suggestion-enable-eol-last-rule-for-eslint-and-ember-template-lint/186471
**Category:** Development
**Created:** [12 באפריל,‏ 2021,‏ 8:46am UTC](https://meta.discourse.org/t/suggestion-enable-eol-last-rule-for-eslint-and-ember-template-lint/186471 "2021-04-12T08:46:13Z")
**Posts on this page:** 3
**Page:** 1

<div class="post-metadata">

### Author: ![fzngagan](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/fzngagan/32/259349_2.png) [@fzngagan](https://meta.discourse.org/u/fzngagan)
#### Post date: [12 באפריל,‏ 2021,‏ 8:46am UTC](https://meta.discourse.org/t/suggestion-enable-eol-last-rule-for-eslint-and-ember-template-lint/186471/1 "2021-04-12T08:46:13Z")

</div>

Its a general dev practice to add a newline at the end of the file as github always complains about it. My suggestion is to enable the `eol-last` rule for eslint and `ember-template-lint`. I see this code here which has it under a comment.

> <https://github.com/discourse/discourse/blob/dc6b547ed89f652b5406489d76140b76cf8e0d1d/.template-lintrc.js#L6>

Thoughts @riking ?

---

<div class="post-metadata">

### Author: ![riking](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/riking/32/170938_2.png) [@riking](https://meta.discourse.org/u/riking)
#### Post date: [12 באפריל,‏ 2021,‏ 5:27pm UTC](https://meta.discourse.org/t/suggestion-enable-eol-last-rule-for-eslint-and-ember-template-lint/186471/2 "2021-04-12T17:27:20Z")

</div>

Oops, I forgot about that one!

I put rules that had too many violations / diffs as “pending” while working on that file. So the way to see the explanation is to enable it and see what gets changed.

---

<div class="post-metadata">

### Author: ![fzngagan](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/fzngagan/32/259349_2.png) [@fzngagan](https://meta.discourse.org/u/fzngagan)
#### Post date: [12 באפריל,‏ 2021,‏ 6:45pm UTC](https://meta.discourse.org/t/suggestion-enable-eol-last-rule-for-eslint-and-ember-template-lint/186471/3 "2021-04-12T18:45:36Z")

</div>

There were only a handlful of errors, so I ended up fixing them and making a PR.  
Also, Am I correct in thinking that the CI used the updated config to run the checks?  
[https://github.com/discourse/discourse/pull/12678](https://github.com/discourse/discourse/pull/12678)
