-
Notifications
You must be signed in to change notification settings - Fork 1
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
An in-range update of nyc is breaking the build 🚨 #46
Comments
After pinning to 14.0.0 your tests are still failing. The reported issue might not affect your project. These imprecisions are caused by inconsistent test results. |
Your tests are still failing with this version. Compare changes CommitsThe new version differs by 9 commits.
See the full diff |
This error seemingly has nothing to do with nyc. It appears there is a breaking change in Node.js 12 in the way that throwing a plain objects (as opposed to an |
Ensures that a test fixture's `verifyAccessDenied` function no longer makes the assumption that the error that is thrown is a plain object. It seems that Node.js 12 modified the `throw` statement to automatically substitute a thrown plain object with an `Error`.
Addressed by #47. |
The devDependency nyc was updated from
14.0.0
to14.1.0
.🚨 View failing branch.
This version is covered by your current version range and after updating it in your project the build failed.
nyc is a devDependency of this project. It might not break your production code or affect downstream projects, but probably breaks your build or test tools, which may prevent deploying or publishing.
Status Details
Commits
The new version differs by 15 commits.
c5d90fa
chore(release): 14.1.0
5cc05f4
chore: Update dependencies
1e39ae1
chore: Refresh snapshots, update test/config-override.js to use helpers (#1085)
3d9eaa4
fix: Purge source-map cache before reporting if cache is disabled. (#1080)
132a074
feat: Add support for env.NYC_CONFIG_OVERRIDE (#1077)
6fc109f
chore: node.js 12 compatibility for object snapshot test. (#1084)
a7bc7ae
fix: Use correct config property for parser plugins (#1082)
600c867
chore: Convert some tap tests to run parallel and use snapshots. (#1075)
56591fa
docs: instrument docs update [skip ci] (#1063)
ca84c42
docs(codecov): favour npx over installing locally [skip ci] (#1074)
85c1eac
chore: Add test for
nyc --no-clean
. (#1071)21fb2c8
fix: Exit with code 1 when nyc doesn't know what to do. (#1070)
0f745ca
chore: Use class to declare NYC (#1069)
ca37ffa
feat: add support for yaml configuration file (#1054)
c4fcf5e
fix: Do not crash when nyc is run inside itself. (#1068)
See the full diff
FAQ and help
There is a collection of frequently asked questions. If those don’t help, you can always ask the humans behind Greenkeeper.
Your Greenkeeper Bot 🌴
The text was updated successfully, but these errors were encountered: