Skip to content
This repository has been archived by the owner on Jan 19, 2019. It is now read-only.

Interface keys can falsely violate no-use-before-define #435

Closed
ncknuna opened this issue Jan 12, 2018 · 1 comment · Fixed by #540
Closed

Interface keys can falsely violate no-use-before-define #435

ncknuna opened this issue Jan 12, 2018 · 1 comment · Fixed by #540

Comments

@ncknuna
Copy link

ncknuna commented Jan 12, 2018

What version of TypeScript are you using?
2.6.2

What version of typescript-eslint-parser are you using?
11.0.0

What code were you trying to parse?

interface Foo {
    bar: string
}

const bar = 'blah'

What did you expect to happen?
The code to be considered valid from the perspective of no-use-before-define, or the README to call out that this is an unsupported case.

What happened?
The key bar in the interface on line 2 was flagged as being used before defined. If I removed the later const bar... assignment, the lint error went away.

@JamesHenry
Copy link
Member

Thanks for reporting @ncknuna!

mysticatea added a commit that referenced this issue Nov 8, 2018
mysticatea added a commit that referenced this issue Nov 13, 2018
* Update: add proper scope analysis (fixes #535)

* add computed-properties-in-type fixture

* add computed-properties-in-interface fixture

* add function-overload fixture

* add method-overload fixture

* add class-properties fixture

* add decorators fixture

* update visitor-keys

* add declare-global fixture

* fix typo

* add test for typeof in array destructuring

* add namespace fixture

* add declare-module fixture

* fix crash

* add declare-function.ts fixture

* add abstract-class fixture

* add typeof-in-call-signature fixture

* add test for #416

* add test for #435

* add test for #437

* add test for #443

* add test for #459

* add test for #466

* add test for #471

* add test for #487

* add test for #535

* add test for #536

* add test for #476

* fix test to use `expect()`
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants