search

stealjs / steal-less

A less plugin for StealJS
https://www.npmjs.com/package/steal-less
MIT License
4 stars 2 forks source link

An in-range update of steal is breaking the build 🚨 #70

Closed greenkeeper[bot] closed 5 years ago

greenkeeper[bot] commented 6 years ago

Version 1.12.3 of steal was just published.

Branch Build failing 🚨
Dependency [steal](https://github.com/stealjs/steal)
Current Version 1.12.2
Type devDependency

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

steal 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 failed [Details](https://travis-ci.org/stealjs/steal-less/builds/394199956?utm_source=github_status&utm_medium=notification)
Release Notes 1.12.3

Resolves a bug when loading the steal package.json from multiple entry points.

Commits

The new version differs by 2 commits.

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 6 years ago

After pinning to 1.12.2 your tests are passing again. Downgrade this dependency 📌.

greenkeeper[bot] commented 6 years ago

Version 1.12.4 just got published.

Your tests are still failing with this version. Compare the changes 🚨

Release Notes 1.12.4

This is a patch release, fixing a bug where a CommonJS module was detected as ES6 because of code inside of a comment.

Bugs

Commits

The new version differs by 3 commits.

  • d867dbd 1.12.4
  • b51263c Merge pull request #1430 from stealjs/react-redux
  • 4ff1605 Make steal ignore ES6 syntax inside of comments

See the full diff

m-mujica commented 5 years ago

@matthewp should steal-less depend on steal@2 instead? does that mean we need a steal-less@2 ?

Anyway I guess we need to figure out why steal@1.x breaks the build.