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.

Sign up for GitHub

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

Jump to bottom

docs: add vitest intro to typescript/testing #5336

Closed
wants to merge 2 commits into from
Closed

docs: add vitest intro to typescript/testing #5336

wants to merge 2 commits into from

Conversation

zxch3n
Copy link

@zxch3n zxch3n commented May 13, 2022

I'm not sure whether a gif screenshot is too bothering. Let me know if you guys prefer code examples or static screenshots ;)

@zxch3n zxch3n changed the title docs: add vitest intro docs: add vitest intro to typescript/testing May 13, 2022
ghost
ghost suggested changes Jul 17, 2022
View reviewed changes
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

I feel like the other ones should be consistent with what you added to Vitest, a Mocha, Playwright, and Jest intro.

ghost

This comment was marked as spam.

Sign in to view

@kenshanta
Copy link

Is this PR still valid? 👀

@ntrogh
Copy link
Contributor

ntrogh commented Dec 17, 2024

Closing this PR as we need to rewrite the content for testing in TypeScript. Current general testing guidance is available here: https://code.visualstudio.com/docs/editor/testing

@ntrogh ntrogh closed this Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cadamini cadamini cadamini approved these changes

@ProKashif ProKashif ProKashif approved these changes

@rondales rondales rondales approved these changes

@asus831 asus831 asus831 approved these changes

@Lo236816 Lo236816 Lo236816 approved these changes

Assignees
No one assigned
Labels
don't merge TypeScript
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@zxch3n @kenshanta @ntrogh @cadamini @ProKashif @rondales @asus831 @Lo236816 @gregvanl