-
-
Notifications
You must be signed in to change notification settings - Fork 0
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
options for strict tests; few enhancements #114
Conversation
Unit Tests Summary 1 files 111 suites 11m 47s ⏱️ Results for commit 6737049. ♻️ This comment has been updated with latest results. |
Unit Test Performance Difference
Additional test case details
Results for commit f829201 ♻️ This comment has been updated with latest results. |
…st into strict_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.
please investigate the cause of the snapshot changes, is it just because of data changing?
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.
Lgtm! We can unlock this, then in the issue #127 we can take care having fully updated snapshots
Thank you! Merging this |
--as-cran
incheck.yaml
andrelease.yaml
to allow snapshot testing.Please review the changes carefully and let me know if there is something you don't like.