-
Notifications
You must be signed in to change notification settings - Fork 198
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
chore(github): enable github actions #277
Conversation
.github/workflows/health-check.yml
Outdated
runs-on: ubuntu-latest | ||
strategy: | ||
matrix: | ||
node: [ '10.x', '12.x', '14.x' ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are these versions meant to be different for the health check vs tests?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nope they should ideally be the same, i have updated them
.github/workflows/tests.yml
Outdated
runs-on: ubuntu-latest | ||
strategy: | ||
matrix: | ||
node: [ '12.x', '14.x', '16.x' ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should probably test on 10 since we still support it and not supporting would be a breaking change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added it back
🎉 This PR is included in version 4.5.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Update to use Github actions as travis.org is now discontinued
Description
Motivation and Context
How Has This Been Tested?
Types of Changes
Checklist:
What is the Impact to Developers Using Jest-Image-Snapshot?