Skip to content

fix CI build - #54386

Merged
Jake Bailey (jakebailey) merged 2 commits into
microsoft:mainfrom
a-tarasyuk:fix/move-to-file-tests
May 25, 2023
Merged

fix CI build#54386
Jake Bailey (jakebailey) merged 2 commits into
microsoft:mainfrom
a-tarasyuk:fix/move-to-file-tests

Conversation

@a-tarasyuk

Copy link
Copy Markdown
Contributor

No description provided.

@jakebailey

Copy link
Copy Markdown
Member

Fwiw I think we do normalize newlines sometimes in fourslash, if that's part of the issue.

@a-tarasyuk

Copy link
Copy Markdown
ContributorAuthor

Jake Bailey (@jakebailey) The issue is to preserve the new line, from the target file. This PR #54164 conflicts with #54358 which was merged before.

@a-tarasyuk
Oleksandr Tarasiuk (a-tarasyuk) marked this pull request as ready for review May 25, 2023 06:14
@a-tarasyukOleksandr Tarasiuk (a-tarasyuk) changed the title fix formattingfix CI buildMay 25, 2023
Comment threadtests/cases/fourslash/moveToFile_overloads1.ts
@sandersn

Copy link
Copy Markdown
Member

Passes on my machine; I'll merge this when CI finishes.

@jakebailey
Jake Bailey (jakebailey) merged commit 1a8c7ae into microsoft:mainMay 25, 2023
@microsoftMicrosoft (microsoft) locked as resolved and limited conversation to collaborators Oct 22, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@a-tarasyuk@jakebailey@sandersn