-
Notifications
You must be signed in to change notification settings - Fork 668
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
feat: Optionally hide deprecation errors #1518
Conversation
What would be the way to do that ? the config |
@fredicious it should work - did you do this:
In your test setup file? It should disable for all the tests. |
that worked, thanks a lot @lmiller1990 |
@@ -1,6 +1,7 @@ | |||
// @flow | |||
import Vue from 'vue' | |||
import semver from 'semver' | |||
import { config } from '@vue/test-utils' |
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.
Not sure why, but this new import here breaks the IIFE. Would love some help tracking this down and fixing it. See #1553
Allows users to hide the deprecation warnings and hides them while our internal tests are run.
What kind of change does this PR introduce? (check at least one)
Does this PR introduce a breaking change? (check one)
If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:
dev
branch.fix #xxx[,#xxx]
, where "xxx" is the issue number)If adding a new feature, the PR's description includes:
Other information: