-
-
Notifications
You must be signed in to change notification settings - Fork 80
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
Add integration test snapshots #905
Add integration test snapshots #905
Conversation
f3b62d3
to
3cf3f81
Compare
CI is failing because it doesn't have |
There's definitely a
Something weird is going on with how it's invoked, I suppose? The code change looks great and I especially love the detailed module-level doc comment. Happy to merge as soon as CI passes. |
I think you might have fixed it. You might just need to set the color=never env var or flag, and update the snapshot. |
That breakage is due to #902, I'm going to disable those tests until we find a fix for the problem. It's not related to your PR. |
Adds functionality using
insta_cmd
to create snapshot tests using thecargo-semver-checks
executable. I added a--bugreport
test as an example.