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

test: rely on Ava default glob #110

Merged
merged 1 commit into from
May 10, 2024
Merged

test: rely on Ava default glob #110

merged 1 commit into from
May 10, 2024

Conversation

turadg
Copy link
Member

@turadg turadg commented May 9, 2024

refs: #8273

Description

This repo is already using the Ava default glob so this PR just removes the vestige that was selecting the old one

Security Considerations

Scaling Considerations

Documentation Considerations

Testing Considerations

@turadg turadg requested a review from samsiegart May 9, 2024 23:08
Copy link

Deploying ui-kit with  Cloudflare Pages  Cloudflare Pages

Latest commit: cf7151b
Status: ✅  Deploy successful!
Preview URL: https://41fa1165.ui-kit-dwm.pages.dev
Branch Preview URL: https://8273-ava-glob.ui-kit-dwm.pages.dev

View logs

@turadg turadg enabled auto-merge (rebase) May 9, 2024 23:30
Copy link
Contributor

@samsiegart samsiegart left a comment

Choose a reason for hiding this comment

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

I think we can actually remove all of ava because all the tests are vitest now

@turadg turadg merged commit ca0ddde into main May 10, 2024
2 checks passed
@turadg turadg deleted the 8273-Ava-glob branch May 10, 2024 00:03
@samsiegart
Copy link
Contributor

I think we can actually remove all of ava because all the tests are vitest now

Whoops didn't see automerge 😄 was just a suggestion though

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.

2 participants