-
Notifications
You must be signed in to change notification settings - Fork 30k
fix types of eslint-config-next #84929
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
Conversation
|
Allow CI Workflow Run
Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer |
|
I just encountered this issue after installing next 16. I was just about to file a PR, but thankfully @davbrito already opened this one. Great job btw! Is there anything we can do to help this get merged more quickly? |
Meanwhile I use bun patch, and for other package managers you can use patch-package. Hope it helps! |
4bf2499 to
e7985d9
Compare
|
This PR fixes #85379 in a cleaner way |
`eslint-config-next` types were being emitted to `dist/src` instead of `dist/` during build. | Before | After | |--------|--------| | <img width="422" height="452" alt="CleanShot 2025-11-04 at 15 50 38@2x" src="https://github.com/user-attachments/assets/09c96d1a-41ba-4da9-87c6-bc05257c6e2a" /> | <img width="386" height="488" alt="CleanShot 2025-11-04 at 15 50 01@2x" src="https://github.com/user-attachments/assets/7b9802a4-e453-4086-90fb-a19e1aec7e20" /> | Fixes #85379 Closes: #84929 Closes: #85653
`eslint-config-next` types were being emitted to `dist/src` instead of `dist/` during build. | Before | After | |--------|--------| | <img width="422" height="452" alt="CleanShot 2025-11-04 at 15 50 38@2x" src="https://github.com/user-attachments/assets/09c96d1a-41ba-4da9-87c6-bc05257c6e2a" /> | <img width="386" height="488" alt="CleanShot 2025-11-04 at 15 50 01@2x" src="https://github.com/user-attachments/assets/7b9802a4-e453-4086-90fb-a19e1aec7e20" /> | Fixes #85379 Closes: #84929 Closes: #85653
`eslint-config-next` types were being emitted to `dist/src` instead of `dist/` during build. | Before | After | |--------|--------| | <img width="422" height="452" alt="CleanShot 2025-11-04 at 15 50 38@2x" src="https://github.com/user-attachments/assets/09c96d1a-41ba-4da9-87c6-bc05257c6e2a" /> | <img width="386" height="488" alt="CleanShot 2025-11-04 at 15 50 01@2x" src="https://github.com/user-attachments/assets/7b9802a4-e453-4086-90fb-a19e1aec7e20" /> | Fixes vercel#85379 Closes: vercel#84929 Closes: vercel#85653
The types were published on
dist/src/*.d.tsbut onpackage.jsonexportspoints todist/*.d.tsThis PR adds
"roorDir": "src"totsconfig.jsonso it builds with the right file structure: