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

feat!: use enhanced-resolve instead #110

Merged
merged 2 commits into from Jun 25, 2022
Merged

feat!: use enhanced-resolve instead #110

merged 2 commits into from Jun 25, 2022

Conversation

JounQin
Copy link
Collaborator

@JounQin JounQin commented Jun 25, 2022

close #85, close #107

@JounQin JounQin self-assigned this Jun 25, 2022
@JounQin JounQin merged commit 7e0cd04 into master Jun 25, 2022
@JounQin JounQin deleted the feat/exports branch June 25, 2022 09:32
ijjk added a commit to vercel/next.js that referenced this pull request Nov 16, 2022
…3010)

<!--
Thanks for opening a PR! Your contribution is much appreciated.
To make sure your PR is handled as smoothly as possible we request that
you follow the checklist sections below.
Choose the right checklist for the change that you're making:
-->

## Bug

Bumps `eslint-import-resolver-typescript` in `eslint-config-next` to the
latest version to pick up a fix from `3.1.0`. As mentioned here,
import-js/eslint-import-resolver-typescript#107,
the typescript parser is unable to resolve the `typesVersions ` field in
their `package.json`. An example repo using this is
[`@vercel/examples-ui`](https://github.com/vercel/examples/blob/385a2e968619f9e50f488e2857e021841ae61030/packages/ui/package.json#L14-L22)
which will report an eslint `import/no-unresolved` error when using the
previous version of `eslint-import-resolver-typescript` (see
import-js/eslint-import-resolver-typescript#107).
This was fixed in
import-js/eslint-import-resolver-typescript#110

This can be locally fixed by bumping
`>=eslint-import-resolver-typescript@3.1.0` but decided it should be
resolved here too.

- [ ] Related issues linked using `fixes #number`
- [ ] Integration tests added
- [x] Errors have a helpful link attached, see `contributing.md`

Closes: #41786

Co-authored-by: JJ Kasper <jj@jjsweb.site>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
1 participant