Skip to content
This repository has been archived by the owner on Mar 25, 2021. It is now read-only.

4.4.1 Unexpected "missing whitespace" errors in end of file #2129

Closed
justtal opened this issue Jan 26, 2017 · 3 comments · Fixed by singapore/lint-condo#223
Closed

4.4.1 Unexpected "missing whitespace" errors in end of file #2129

justtal opened this issue Jan 26, 2017 · 3 comments · Fixed by singapore/lint-condo#223

Comments

@justtal
Copy link

justtal commented Jan 26, 2017

Bug Report

  • TSLint version: 4.4.1
  • TypeScript version: 2.1.5
  • Running TSLint via: CLI

TypeScript code being linted

any .ts file

with tslint.json configuration:

{
  "rules": {
    "quotemark": [
      true,
      "single",
      "avoid-escape"
    ],
    "class-name": true,
    "no-consecutive-blank-lines": true,
    "semicolon": [
      true,
      "always"
    ],
    "whitespace": [
      true,
      "check-branch",
      "check-decl",
      "check-operator",
      "check-module",
      "check-separator",
      "check-type"
    ],
    "no-var-keyword": true,
    "interface-name": [true, "always-prefix"],
    "adjacent-overload-signatures": true,
    "use-isnan": true,
    "one-variable-per-declaration": true
  }
}

Actual behavior

Unexpected "missing whitespace" errors in all files (in the last char of the file)

Expected behavior

No unexpected errors

Thanks!

@nchen63
Copy link
Contributor

nchen63 commented Jan 26, 2017

I can't reproduce. can you put it in a zip file and attach it? Also, what OS?

@justtal
Copy link
Author

justtal commented Jan 26, 2017

OS X Yosemite.

mock.enum.ts.zip

In version 4.3.1 it doesn't happen.

@larshp
Copy link
Contributor

larshp commented Jan 26, 2017

I have the same issue, see abaplint/abaplint#134, it does not happen in 4.3.1

it looks like it is related to the check-separator option, guess EOF after ; is not regarded as whitespace anymore

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants