Content-Length: 294532 | pFad | https://github.com/typescript-eslint/typescript-eslint/issues/5856

4A Refactor: Remove createIsolatedProgram call from getProgramAndAST · Issue #5856 · typescript-eslint/typescript-eslint · GitHub
Skip to content

Refactor: Remove createIsolatedProgram call from getProgramAndAST #5856

@JoshuaKGoldberg

Description

@JoshuaKGoldberg

Overview

createDefaultProgram(code, extra)) ||
createIsolatedProgram(code, extra)

That last || createIsolatedProgram(code, extra) should never be hit. We believe it was a just-in-case fallback added long ago that is no longer (and maybe never was) needed.

Removing that case may make #5855 easier.

Marking as accepting PRs with the caveat that everything still has to work as expected. Which may or may not be doable. Good luck! 😄

This issue is from a pairing I had with @bradzacher going over the parser internals.

Metadata

Metadata

Assignees

No one assigned

    Labels

    accepting prsGo ahead, send a pull request that resolves this issuebreaking changeThis change will require a new major version to be releasedlocked due to agePlease open a new issue if you'd like to say more. See https://typescript-eslint.io/contributing.refactorPRs that refactor code only

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions









      ApplySandwichStrip

      pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


      --- a PPN by Garber Painting Akron. With Image Size Reduction included!

      Fetched URL: https://github.com/typescript-eslint/typescript-eslint/issues/5856

      Alternative Proxies:

      Alternative Proxy

      pFad Proxy

      pFad v3 Proxy

      pFad v4 Proxy