Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

@typescript-eslint/no-use-before-define #2941

Closed
GuoSirius opened this issue Jan 19, 2021 · 4 comments · Fixed by #2943
Closed

@typescript-eslint/no-use-before-define #2941

GuoSirius opened this issue Jan 19, 2021 · 4 comments · Fixed by #2943
Labels
bug Something isn't working package: eslint-plugin Issues related to @typescript-eslint/eslint-plugin

Comments

@GuoSirius
Copy link

class A {
  constructor (printName) {
    this.printName = printName
  }

  openPort(printerName = this.printerName) {
    this.tscOcx.ActiveXopenport(printerName)

    return this
  }
}

image

image

@GuoSirius GuoSirius added package: eslint-plugin Issues related to @typescript-eslint/eslint-plugin triage Waiting for maintainers to take a look labels Jan 19, 2021
@GuoSirius
Copy link
Author

import { Vue, Component } from 'vue-property-decorator'

@Component({
  computed: {}
})
export default class NotFound extends Vue {}

image

@bradzacher
Copy link
Member

bradzacher commented Jan 19, 2021

The second issue is for the no-undef rule, not the no-use-before-define rule - #2942

@bradzacher bradzacher added awaiting response Issues waiting for a reply from the OP or another party bug Something isn't working and removed triage Waiting for maintainers to take a look awaiting response Issues waiting for a reply from the OP or another party labels Jan 19, 2021
@bradzacher
Copy link
Member

Same cause as #2942 - a regression introduced in a recent update in the scope analyser.

bradzacher added a commit that referenced this issue Jan 19, 2021
…ass method default params

Fixes #2941
Fixes #2942

This was a regression introduced by #2751
bradzacher added a commit that referenced this issue Jan 19, 2021
…ass method default params (#2943)

Fixes #2941
Fixes #2942

This was a regression introduced by #2751
@paztis
Copy link

paztis commented Jan 19, 2021

Facing this issue.
is it possible to release the fix in 4.14.1 ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working package: eslint-plugin Issues related to @typescript-eslint/eslint-plugin
Projects
None yet
3 participants