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

Update workflow #120

Closed
wants to merge 8 commits into from
Closed

Update workflow #120

wants to merge 8 commits into from

Conversation

m2rads
Copy link
Collaborator

@m2rads m2rads commented Dec 19, 2024

The Vitest tests are not being maintained and the Vitest workflow keeps failing.

This PR removes the Vites workflow and enhances Shortest workflow to be more robust in executing the tests. The aim is to bring the branch to a healthy state for better open source collaboration.

@m2rads m2rads requested review from slavingia and rmarescu December 19, 2024 10:47
Copy link

vercel bot commented Dec 19, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
shortest ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 19, 2024 11:04am

@m2rads
Copy link
Collaborator Author

m2rads commented Dec 19, 2024

@slavingia, @rmarescu
Maybe we can also remove all the vitests or should we keep them?

Updating our vitest test and enhancing the web app to have better Vitest test generation is another option we can work on.

@m2rads m2rads closed this Dec 20, 2024
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