8000 Enhancement: Default projectService.defaultProject to "tsconfig.json" · Issue #9807 · typescript-eslint/typescript-eslint · GitHub
[go: up one dir, main page]
Skip to content

Enhancement: Default projectService.defaultProject to "tsconfig.json" #9807

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.

Sign up for GitHub

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

Jump to bottom
Closed
8000 4 tasks done
JoshuaKGoldberg opened this issue Aug 16, 2024 · 0 comments · Fixed by #9893
Closed
4 tasks done

Enhancement: Default projectService.defaultProject to "tsconfig.json" #9807

JoshuaKGoldberg opened this issue Aug 16, 2024 · 0 comments · Fixed by #9893
Labels
accepting prs Go ahead, send a pull request that resolves this issue enhancement New feature or request locked due to age Please open a new issue if you'd like to say more. See https://typescript-eslint.io/contributing. package: typescript-estree Issues related to @typescript-eslint/typescript-estree

Comments

@JoshuaKGoldberg
Copy link
Member

Before You File a Proposal Please Confirm You Have Done The Following...

Relevant Package

parser

My proposal is suitable for this project

  • I believe my proposal would be useful to the broader TypeScript community (meaning it is not a niche proposal).

Description

Right now, if you want to use parserOptions.projectService's allowDefaultProject, you need to also specify a defaultProject alongside it. But that default project is basically always 'tsconfig.json' as I've set it.

As a convenience for users, how about we default it to 'tsconfig.json' instead of undefined?

Additional Info

Resolution to this issue -either a PR or a wontfix- should block #8031 IMO. If we do end up making the change then the blog post will need to be rephrased a bit.

💖
@JoshuaKGoldberg JoshuaKGoldberg added enhancement New feature or request triage Waiting for team members to take a look labels Aug 16, 2024
@JoshuaKGoldberg JoshuaKGoldberg mentioned this issue Aug 19, 2024
Closed
4 tasks
@bradzacher bradzacher added accepting prs Go ahead, send a pull request that resolves this issue package: typescript-estree Issues related to @typescript-eslint/typescript-estree an 8608 d removed triage Waiting for team members to take a look labels Aug 25, 2024
@JoshuaKGoldberg JoshuaKGoldberg mentioned this issue Aug 27, 2024
Merged
3 tasks
@github-actions github-actions bot added the locked due to age Please open a new issue if you'd like to say more. See https://typescript-eslint.io/contributing. label Sep 14, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
accepting prs Go ahead, send a pull request that resolves this issue enhancement New feature or request locked due to age Please open a new issue if you'd like to say more. See https://typescript-eslint.io/contributing. package: typescript-estree Issues related to @typescript-eslint/typescript-estree
Projects
None yet
Milestone
No milestone
2 participants
@JoshuaKGoldberg @bradzacher
0