Skip to content

fix type import check for default-import/re-export in js files - #57778

Merged
Andrew Branch (andrewbranch) merged 1 commit into
microsoft:mainfrom
jasonlyu123:auto-import-aliased-js
Mar 27, 2024
Merged

Andrew Branch (andrewbranch) merged 1 commit into
microsoft:mainfrom
jasonlyu123:auto-import-aliased-js

Conversation

@jasonlyu123

@jasonlyu123 Lyu, Wei-Da (jasonlyu123) commented Mar 14, 2024 •

Copy link
Copy Markdown
Contributor

Fixes #57740. Pretty much copied from shouldIncludeSymbol. getCombinedLocalAndExportSymbolFlags for the default export and skipAlias for export { someFunciton } and export { someFunction } from './module'.

@andrewbranch
Andrew Branch (andrewbranch) merged commit b0d5ae6 into microsoft:main Mar 27, 2024
@andrewbranch

Copy link
Copy Markdown
Member

Thanks Lyu, Wei-Da (@jasonlyu123)!

@andrewbranch

Copy link
Copy Markdown
Member

TypeScript Bot (@typescript-bot) cherry-pick to release-5.4

@typescript-bot

TypeScript Bot (typescript-bot) commented Apr 8, 2024 •

Copy link
Copy Markdown
Contributor

Starting jobs; this comment will be updated as builds start and complete.

Command Status Results
cherry-pick to release-5.4 ✅ Started ✅ Results

@typescript-bot

Copy link
Copy Markdown
Contributor

Hey, Andrew Branch (@andrewbranch)! I've created #58116 for you.

Daniel Rosenwasser (DanielRosenwasser) pushed a commit that referenced this pull request Apr 8, 2024
…e-5.4 (#58116)

Co-authored-by: Lyu, Wei-Da <36730922+jasonlyu123@users.noreply.github.com>
@microsoft Microsoft (microsoft) locked as resolved and limited conversation to collaborators Oct 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

For Backlog Bug PRs that fix a backlog bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

No auto-import from aliased symbol from ts files in js files

6 participants