Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: typescript-eslint/typescript-eslint
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v4.7.0
Choose a base ref
...
head repository: typescript-eslint/typescript-eslint
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v4.8.0
Choose a head ref
  • 7 commits
  • 32 files changed
  • 5 contributors

Commits on Nov 10, 2020

  1. Configuration menu
    Copy the full SHA
    e2a15f1 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    33ce420 View commit details
    Browse the repository at this point in the history

Commits on Nov 11, 2020

  1. test(typescript-estree): [babel alignment] remove now unnecessary AST…

    … transformation for ImportDeclaration (#2754)
    sosukesuzuki authored Nov 11, 2020
    Configuration menu
    Copy the full SHA
    abe33fb View commit details
    Browse the repository at this point in the history
  2. fix(eslint-plugin): [no-unsafe-member-access] ignore MemberExpression…

    …'s whose parents are either TSClassImplements or TSInterfaceHeritage (#2753)
    hsmitty93 authored Nov 11, 2020
    Configuration menu
    Copy the full SHA
    535db3b View commit details
    Browse the repository at this point in the history

Commits on Nov 12, 2020

  1. feat(typescript-estree): add parseWithNodeMaps API (#2760)

    Ref: prettier/prettier#9636
    
    This allows consumers to reach into the underlying TS AST in cases where our AST doesn't quite solve the use case.
    
    Motivating example:
    
    We don't (currently) error for code unless TS itself throws an error.
    TS is _very_ permissive, but that leads to some weird (and invalid) code we don't error for, and don't include in the AST.
    For example - this is _syntactically_ valid in the TS parser, but they emit a _semantic_ error:
    ```ts
    @decorator
    enum Foo {
      A = 1
    }
    ```
    As it's not a valid place for a decorator - we do not emit its information in the ESTree AST, but as TS treats this as a semantic error - it is parse-time valid.
    This creates weird states for consumers wherein they cannot see a decorator exists there, which can cause them to ignore it entirely.
    For linting - this isn't a problem. But for something like prettier - this means that they'll delete the decorator at format time!
    This is very bad.
    
    Until we implement a solution for #1852 - this will allow consumers to bridge the gap themselves.
    bradzacher authored Nov 12, 2020
    Configuration menu
    Copy the full SHA
    9441d50 View commit details
    Browse the repository at this point in the history

Commits on Nov 14, 2020

  1. fix(eslint-plugin): [consistent-type-definitions] remove fixer when t…

    …he interface is within a global module declaration (#2739)
    Daniil Dubrava authored Nov 14, 2020
    Configuration menu
    Copy the full SHA
    2326238 View commit details
    Browse the repository at this point in the history

Commits on Nov 16, 2020

  1. chore: publish v4.8.0

    JamesHenry committed Nov 16, 2020
    Configuration menu
    Copy the full SHA
    3333040 View commit details
    Browse the repository at this point in the history
Loading