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

test: migrate from jest to vitest #90

Merged
merged 3 commits into from
Mar 3, 2024
Merged

test: migrate from jest to vitest #90

merged 3 commits into from
Mar 3, 2024

Conversation

AlbertHernandez
Copy link
Owner

@AlbertHernandez AlbertHernandez commented Mar 3, 2024 •
edited
Loading

Proposed changes

The objective of this PR is to migrate from jest to vitest.

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • Any dependent changes have been merged and published in downstream modules

Albert Hernandez Pellicer added 2 commits March 3, 2024 13:51
@github-actions github-actions bot added ⚙️ Configuration Indicates the scope is related to the configuration 📦 Dependencies Pull requests that update a dependency file 🐳 Build Indicates the change is related to the build 🧪 Tests Indicates the scope is related to the tests 📝 Documentation Improvements or additions to documentation 🚀 CI/CD Indicates the change is related to CI/CD workflows labels Mar 3, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 3, 2024 •
edited
Loading

Dependency Review

✅ No vulnerabilities or license issues found.

Scanned Manifest Files

package-lock.json
package.json
  • @vitest/coverage-istanbul@^1.3.1
  • eslint-plugin-vitest@^0.3.22
  • unplugin-swc@^1.4.4
  • vite@^5.1.4
  • vitest@^1.3.1
  • @swc/jest@^0.2.36
  • @types/jest@^29.5.12
  • eslint-plugin-jest@^27.9.0
  • jest@^29.7.0
  • ts-jest@^29.1.2

@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 3, 2024

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@github-actions github-actions bot added the 🤯 size/xl Pull request size XL label Mar 3, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 3, 2024 •
edited
Loading

🐳 Docker Metrics 🐳

  • Size of the Docker Image in the base (main): 139MB
  • Size of the Docker Image in this branch (test/migrate-from-jest-to-vitest): 139MB

@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 3, 2024

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@AlbertHernandez AlbertHernandez merged commit d592959 into main Mar 3, 2024
23 checks passed
@AlbertHernandez AlbertHernandez deleted the test/migrate-from-jest-to-vitest branch March 3, 2024 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@AlbertHernandez AlbertHernandez

Labels
🐳 Build Indicates the change is related to the build 🚀 CI/CD Indicates the change is related to CI/CD workflows ⚙️ Configuration Indicates the scope is related to the configuration 📦 Dependencies Pull requests that update a dependency file 📝 Documentation Improvements or additions to documentation 🤯 size/xl Pull request size XL 🧪 Tests Indicates the scope is related to the tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@AlbertHernandez