-
Notifications
You must be signed in to change notification settings - Fork 768
added gating for user preferences for auto imports #1793
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
…3/typescript-go into auto-imports-user-prefs" This reverts commit 865d44bc8948e7165defb117837464f5cbe2326d, reversing changes made to 186540b5240e2e31e61a2c4a8467d79b8174292c.
|
Is the test generator just missing these options such that all tests are new handwritten ones? |
Isabel mentioned that I'd need to handwrite tests using |
|
Codefixes were ported in #2053 |
jakebailey
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems good, just a few small things.
| return true | ||
| } | ||
| } | ||
| // !! SymlinkCache |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The symlink cache should now be available.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't bother changing anything in this file more than you need to
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
alright, ignore me, I guess this is going away soon 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good, should be ready for final review then
…refs' into auto-imports-user-prefs
Implements and tests User Preferences for Auto Imports; the specific preferences included are shown below.
importModuleSpecifierPreference: "shortest" | "project-relative" | "relative" | "non-relative";importModuleSpecifierEnding: "auto" | "minimal" | "index" | "js";includePackageJsonAutoImports: "auto" | "on" | "off";allowRenameOfImportPath: boolean;autoImportFileExcludePatterns: string[];autoImportSpecifierExcludeRegexes: string[];preferTypeOnlyAutoImports: boolean;