Skip to content

If import is used in the file, prefer that import specifier over calculating new one - #42224

Merged
Sheetal Nandi (sheetalkamat) merged 7 commits into
masterfrom
projectRefDeclarationEmit
Jan 8, 2021
Merged

If import is used in the file, prefer that import specifier over calculating new one#42224
Sheetal Nandi (sheetalkamat) merged 7 commits into
masterfrom
projectRefDeclarationEmit

Conversation

@sheetalkamat

@sheetalkamatSheetal Nandi (sheetalkamat) commented Jan 5, 2021

Copy link
Copy Markdown
Member

Fixes#39117

@typescript-botTypeScript Bot (typescript-bot) added Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug labels Jan 5, 2021
@sheetalkamatSheetal Nandi (sheetalkamat) changed the title Project ref declaration emitIf import is used in the file, prefer that import specifier over calculating new oneJan 5, 2021
@typescript-botTypeScript Bot (typescript-bot) added For Milestone Bug PRs that fix a bug with a specific milestone and removed For Uncommitted Bug PR for untriaged, rejected, closed or missing bug labels Jan 5, 2021
Comment threadsrc/compiler/moduleSpecifiers.ts
Comment threadsrc/compiler/moduleSpecifiers.ts
@sheetalkamat

Copy link
Copy Markdown
MemberAuthor

@typescript-bot

TypeScript Bot (typescript-bot) commented Jan 6, 2021

Copy link
Copy Markdown
Contributor

Heya Sheetal Nandi (@sheetalkamat), I've started to run the parallelized community code test suite on this PR at 71e42ba. You can monitor the build here.

@typescript-bot

TypeScript Bot (typescript-bot) commented Jan 6, 2021

Copy link
Copy Markdown
Contributor

Heya Sheetal Nandi (@sheetalkamat), I've started to run the extended test suite on this PR at 71e42ba. You can monitor the build here.

@typescript-bot

Copy link
Copy Markdown
Contributor

The user suite test run you requested has finished and failed. I've opened a PR with the baseline diff from master.

@sheetalkamat
Sheetal Nandi (sheetalkamat) deleted the projectRefDeclarationEmit branch January 8, 2021 23:20
Zzzen (Zzzen) pushed a commit to Zzzen/TypeScript that referenced this pull request Jan 16, 2021
…ulating new one (microsoft#42224)
* Test case where the wrong path is emitted
* If import is used in the file, prefer that import specifier over calculating new one
Fixesmicrosoft#39117
* Update Baselines and/or Applied Lint Fixes
* When non-relative path is used as user preference, ignore relative paths even if they are from the existing file
* Fix test
* Add comment
Co-authored-by: TypeScript Bot <typescriptbot@microsoft.com>
@microsoftMicrosoft (microsoft) locked as resolved and limited conversation to collaborators Oct 21, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Author: TeamFor Milestone BugPRs that fix a bug with a specific milestone

Projects

None yet

Development

Successfully merging this pull request may close these issues.

With project references, some cross-package imports of inferred types are emitted with relative paths

4 participants

@sheetalkamat@typescript-bot@andrewbranch@robpalme