Skip to content
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

fix(58955): @import JSDoc tag doesn't seem to account for resolution-mode import attribute in TS v5.5.2 #58966

Merged
merged 2 commits into from
Jun 24, 2024

Conversation

a-tarasyuk
Copy link
Contributor

Fixes #58955

@typescript-bot typescript-bot added the For Milestone Bug PRs that fix a bug with a specific milestone label Jun 21, 2024
Copy link
Member

@jakebailey jakebailey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems correct to me. Probably worth the 5.5 backport.

Copy link
Member

@andrewbranch andrewbranch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @a-tarasyuk!

@andrewbranch
Copy link
Member

@typescript-bot cherry-pick to release-5.5

@typescript-bot
Copy link
Collaborator

typescript-bot commented Jun 24, 2024

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

Command Status Results
cherry-pick to release-5.5 ✅ Started ❌ Results

@typescript-bot
Copy link
Collaborator

Hey, @andrewbranch! I was unable to cherry-pick this PR.

Check the logs at: https://github.com/microsoft/TypeScript/actions/runs/9650506935

@jakebailey
Copy link
Member

I expect cherry-picking to fail just because these lines are ones that changed in the 5.5 bump for type predicate inference...

@a-tarasyuk
Copy link
Contributor Author

@jakebailey should I make a new PR to the release branch?

@jakebailey
Copy link
Member

No worries, I'll do it.

@jakebailey jakebailey merged commit c219989 into microsoft:main Jun 24, 2024
28 checks passed
jakebailey pushed a commit to jakebailey/TypeScript that referenced this pull request Jun 24, 2024
DanielRosenwasser pushed a commit that referenced this pull request Jun 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
For Milestone Bug PRs that fix a bug with a specific milestone
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

@import JSDoc tag doesn't seem to account for resolution-mode import attribute in TS v5.5.2
4 participants