d-band / better-xlsx

A better xlsx library.
https://d-band.github.io/better-xlsx/
420 stars 36 forks source link

An in-range update of rollup is breaking the build 🚨 #77

Closed greenkeeper[bot] closed 5 years ago

greenkeeper[bot] commented 5 years ago

The devDependency rollup was updated from 0.67.1 to 0.67.2.

🚨 View failing branch.

This version is covered by your current version range and after updating it in your project the build failed.

rollup 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 - ❌ **continuous-integration/travis-ci/push:** The Travis CI build could not complete due to an error ([Details](https://travis-ci.org/d-band/better-xlsx/builds/456280728?utm_source=github_status&utm_medium=notification)). - βœ… **coverage/coveralls:** First build on greenkeeper/rollup-0.67.2 at 95.876% ([Details](https://coveralls.io/builds/20157555)).

Commits

The new version differs by 3 commits.

  • fe92d1f 0.67.2
  • ed22964 Update changelog
  • dbd2e47 Fix crash when returning undefined sourcemaps from renderChunk hook (#2558)

See the full diff

FAQ and help There is a collection of [frequently asked questions](https://greenkeeper.io/faq.html). If those don’t help, you can always [ask the humans behind Greenkeeper](https://github.com/greenkeeperio/greenkeeper/issues/new).

Your Greenkeeper Bot :palm_tree:

greenkeeper[bot] commented 5 years ago

After pinning to 0.67.1 your tests are still failing. The reported issue might not affect your project. These imprecisions are caused by inconsistent test results.

greenkeeper[bot] commented 5 years ago

Your tests are still failing with this version. Compare changes

Commits

The new version differs by 4 commits.

  • d9a7d65 0.67.3
  • 4be8888 Update changelog
  • bbdf8b3 Add more files to npm distribution (#2552)
  • 084354e fix: experimentalPreserveModules not supported in inlineDynamicImports message (#2560)

See the full diff