[BUG] It always gives "error Newline required at end of file but not found eol-last"

[trusktr]

No matter how the file ends, it always results in

error  Newline required at end of file but not found  eol-last

For example, I've tried a file ending with these of last lines, and they all result in the same error:

<span>blah</span>
<script>
console.log('hello')
</script>

<span>blah</span>
<script>
console.log('hello')

</script>

<span>blah</span>
<script>
console.log('hello')
</script>

In reality, I have lots more in my file, but it ends with a script tag, and no matter how I modify the last few lines (similarly to the examples) I always get that error.

I can simply disable eol-last for now.

[davidtheclark]

@trusktr: Thanks for the issue! We've been using this plugin with a config tailored to this context, so eol-last has been disabled. It would certainly be better if that wasn't necessary, though! Open to a PR fixing this limitation.

[stalkerg]

I can simply disable eol-last for now.

I am too but I have also a lot of classic .js files and I want to check this.

[trusktr]

@stalkerg Make two different eslintrc files.

Use this file for .js files, .eslintrc:

{
  ... your config here, with eol-last turned on ...
}

then use a separate file for non-.js files that extends the original one, f.e. .eslintrc-script-tag:

{
  "extends": "./.eslintrc",
  "rules": {
    "eol-last": 0,
  }
}

In the arguments to the eslint-plugin-script-tags plugin, tell it to use the second eslintrc.

For example, in a Gulp project, I have

    // ...
    gulp.src(['**/*.hbs'])
      .pipe(eslint({
        plugins: ['script-tags'],
        configFile: './.eslintrc-script-tag'
      }))
      // ...

[fdhcastillo]

Adding a blank line at the end of the file/code worked for me.

[ma-za-kpe]

Adding a blank line at the end of the file/code worked for me.

This works for me too.

[Anastasia-Kraeva]

my problem was setting up line endings in the file
(it was CR instead of LF)