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!: default glob #3729

Merged
merged 7 commits into from
Jul 6, 2023
Merged

fix!: default glob #3729

merged 7 commits into from
Jul 6, 2023

Conversation

so1ve
Copy link
Contributor

@so1ve so1ve commented Jul 3, 2023

close #3530

@netlify
Copy link

netlify bot commented Jul 3, 2023

Deploy Preview for fastidious-cascaron-4ded94 ready!

Name Link
🔨 Latest commit 343b334
🔍 Latest deploy log https://app.netlify.com/sites/fastidious-cascaron-4ded94/deploys/64a52e0e3598180008cb5a2a
😎 Deploy Preview https://deploy-preview-3729--fastidious-cascaron-4ded94.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@so1ve so1ve changed the title fix: default glob fix!: default glob Jul 3, 2023
Copy link
Member

@antfu antfu left a comment

Choose a reason for hiding this comment

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

Looks good to me. I think we should have this as soon before the original breaking change gets spread wide.

As mentioned in #3530 (comment), I do think we should also improve our documentation on the Jest migration, it can be another PR tho.

@antfu
Copy link
Member

antfu commented Jul 5, 2023

@so1ve any thing missing from making it ready to review?

@so1ve so1ve marked this pull request as ready for review July 5, 2023 08:46
@so1ve
Copy link
Contributor Author

so1ve commented Jul 5, 2023

@antfu I think it is ready for review now!

@sheremet-va
Copy link
Member

Looks good to me. I think we should have this as soon before the original breaking change gets spread wide.

I'm fine with releasing 0.33.0 with just this change.

@sheremet-va sheremet-va merged commit f07ef61 into vitest-dev:main Jul 6, 2023
@so1ve so1ve deleted the patch-1 branch July 6, 2023 14:17
@so1ve
Copy link
Contributor Author

so1ve commented Jul 6, 2023

...Forgot to use squash merge?

@sheremet-va
Copy link
Member

...Forgot to use squash merge?

Hehe, yeah

@antfu antfu mentioned this pull request Jul 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

src/test.ts is picked up as a test
3 participants