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

Fix handling of tsconfig's #632

Merged
merged 3 commits into from Nov 3, 2021
Merged

Fix handling of tsconfig's #632

merged 3 commits into from Nov 3, 2021

Conversation

spence-s
Copy link
Contributor

@spence-s spence-s commented Nov 1, 2021

@sindresorhus
Copy link
Member

Seems to fail on Node.js 16 in this TS project: https://github.com/sindresorhus/ow/runs/4091470790?check_suite_focus=true

sindresorhus/ow@db0eef8

@spence-s
Copy link
Contributor Author

spence-s commented Nov 3, 2021

@sindresorhus - I don't think that failure is related to this PR - its a module resolution error. Nothing in this PR changed any module resolution stuff. That failure was from

config.baseConfig.extends.push(require.resolve('eslint-config-xo-typescript'));
- I think this is more likely related to #624

@sindresorhus sindresorhus changed the title fix: properly handle tsconfigs Fix handling of tsconfig's Nov 3, 2021
@sindresorhus sindresorhus merged commit 261696e into xojs:main Nov 3, 2021
@sindresorhus
Copy link
Member

Thanks for fixing this 🙌

@spence-s spence-s deleted the master branch November 3, 2021 17:31
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.

"parserOptions.project" errors occur when users tsconfig does not include a file
2 participants