You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Quite surprised about react showing up that much, given the files I opened all contained an import of react anyway...
Yes that is really surprising. For path references there is code that specifically excludes references to imported files. Guess for type reference directives that logic is missing:
Looking at these, I don’t feel too worried about the volume of real-world breaks being very high. The only thing I might send a preemptive PR for is adding an explicit reference to "../../env.d.ts" to vercel/swr. That said, knowing you need to do that still feels like a footgun. swr’s tsconfig.json explicitly includes env.d.ts. That could be considered bad practice now, as it sets up a program graph that a library consumer would never have. Libraries should probably set their files array to be just their package.json entrypoints. But even that’s unrealistic, since a consumer might not load all entrypoints. Ideally, you’d want to make sure your library code compiles when setting each package.json entrypoint as the lone tsconfig.json files entry. But nobody is going to do that.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Experiment for follow-up discussions from #57440, #57568