stencila / dockta

🐳 A Docker image builder for researchers

Home Page:https://stencila.github.io/dockta/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

An in-range update of fast-glob is breaking the build 🚨

greenkeeper opened this issue Β· comments

The dependency fast-glob was updated from 2.2.6 to 2.2.7.

🚨 View failing branch.

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

fast-glob is a direct dependency of this project, and it is very likely causing it to break. If other packages depend on yours, this update is probably also breaking those in turn.

Status Details
  • ❌ continuous-integration/travis-ci/push: The Travis CI build could not complete due to an error (Details).

Release Notes for 2.2.7

Another release of bug fixes

πŸ“– Documentation

  • Added description of how to work with UNC paths (#89)
  • The ignore option takes an array (#184 β€” thanks @lukeis for contributing)
  • Clarify description of the case option.

πŸ› Bug Fixes

Paths not resolved in some cases (#157)

Thanks @stevenvachon for issue reporting πŸŽ‰

If the user has passed a . or .. and the absolute option is enabled,
the paths of the found entries were not absolute (they contained . or `..).

before

fg.sync('/project/temp/../*.js', { absolute: true }); // β†’ ['/project/temp/../something.js']

after

fg.sync('/project/temp/../*.js', { absolute: true }); // β†’ ['/project/something.js']

The case option not work with static patterns (#172)

Thanks @davidmerfield for issue reporting πŸŽ‰

For performance reasons with fast-glob@2.1.0 we introduce static patterns (patterns without glob magic).

Unfortunately, then we forgot about supporting the case (nocase) option. Now the case option works fine with static patterns too. We also improved the documentation for this option.

directory/
  - file.txt
  - File.txt

before

fg.sync('file.txt', { case: false }) // β†’ ['file.txt']

after

fg.sync('file.txt', { case: false }) // β†’ ['file.txt', 'File.txt']

Question mark is not recognized as dynamic glob and fails to find files (#174)

Thanks @vladshcherbin for issue reporting and contributing πŸŽ‰

This is also related to static patterns.

Previously we mark patterns like assets/?ss.css to static and tried to find such file on file system. Now it will works fine.

before

fg.sync('assets/?ss.css'); // β†’ []

after

fg.sync('assets/?ss.css'); // β†’ ['asserts/css.css']
Commits

The new version differs by 21 commits.

  • 85e42c3 build(package): bump version to 2.2.7
  • a0af28e docs(readme): how to use UNC path
  • 3ea4d8e fix(tasks): always mark patterns as dynamic with case-insensitive mode
  • b22e7be fix(providers): resolve path even for absolute filepath
  • 319b64d Add test for question mark glob
  • 7f259e6 Set is-glob strict to false
  • 62c0fbf Merge pull request #164 from mrmlnc/ISSUE-151_improve_documentation_about_entry_type
  • e7315d7 Merge pull request #165 from mrmlnc/TRIVIAL_update_deps
  • 52cec04 docs(README): clarify return type (fix #151)
  • 8b725de build: drop CI for Node.js 4
  • c41fe33 build: fix build issue after update dependencies
  • 5000834 build: update @types/node
  • 2ed70e8 style(index): use T[] instead of Array<T>
  • 5376d66 Merge pull request #150 from mrmlnc/update_tslint_config
  • a0bb8dd build: update "tslint-config-mrmlnc" to 2.0.0

There are 21 commits in total.

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 🌴

After pinning to 2.2.6 your tests are passing again. Downgrade this dependency πŸ“Œ.