Skip to content

Fix verify.baselineCompletions used in fourslash tests - #45615

Merged
Daniel Rosenwasser (DanielRosenwasser) merged 1 commit into
microsoft:mainfrom
petamoriken:fix/fourslash-baseline-completions
Aug 30, 2021
Merged

Fix verify.baselineCompletions used in fourslash tests#45615
Daniel Rosenwasser (DanielRosenwasser) merged 1 commit into
microsoft:mainfrom
petamoriken:fix/fourslash-baseline-completions

Conversation

@petamoriken

@petamoriken Kenta Moriuchi (petamoriken) commented Aug 28, 2021

Copy link
Copy Markdown
Contributor

Related #43709

I found while working on #43710 that the verify.baselineCompletions used in the fourslash tests were not working properly. This pull request is to fix it.

#43710 (comment)

@typescript-bot

Copy link
Copy Markdown
Contributor

This PR doesn't have any linked issues. Please open an issue that references this PR. From there we can discuss and prioritise.

@petamoriken

Copy link
Copy Markdown
Contributor Author

Nathan Shively-Sanders (@sandersn) Please review this code.

@DanielRosenwasser
Daniel Rosenwasser (DanielRosenwasser) merged commit 9d81536 into microsoft:main Aug 30, 2021
@DanielRosenwasser

Copy link
Copy Markdown
Member

Thanks!

@petamoriken
Kenta Moriuchi (petamoriken) deleted the fix/fourslash-baseline-completions branch August 30, 2021 07:49
Cassandra Choi (BobobUnicorn) pushed a commit to BobobUnicorn/TypeScript that referenced this pull request Oct 24, 2021
@microsoft Microsoft (microsoft) locked as resolved and limited conversation to collaborators Oct 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

For Uncommitted Bug PR for untriaged, rejected, closed or missing bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants