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

Update parserOptions.project for TypeScript files to be true #98

Merged
merged 1 commit into from
Apr 24, 2023

Conversation

kjots
Copy link
Contributor

@kjots kjots commented Apr 24, 2023

This change causes the typescript-eslint plugin to use the tsconfig.json file nearest to file being linted, rather than the tsconfig.json file located in the directory from which ESLint is launched.

Without this change ESLint can break when multiple tsconfig.json files are present (e.g. in a monorepo) and a single file is being linted in isolation, especially in IDEs such as IntelliJ which will always launch ESLint from the project root directory.

More info can be found at:

@changeset-bot
Copy link

changeset-bot bot commented Apr 24, 2023

🦋 Changeset detected

Latest commit: c306d26

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
eslint-config-seek Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@kjots kjots added this pull request to the merge queue Apr 24, 2023
Merged via the queue into master with commit 95b6f78 Apr 24, 2023
@kjots kjots deleted the typescript-parser-options-project branch April 24, 2023 01:39
@seek-oss-ci seek-oss-ci mentioned this pull request Apr 24, 2023
72636c added a commit to seek-oss/eslint-config-skuba that referenced this pull request May 1, 2023
72636c added a commit to seek-oss/eslint-config-skuba that referenced this pull request May 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants