Skip to content
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

[email protected] breaks build 🚨 #126

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

greenkeeperio-bot
Copy link
Contributor

Hello lovely humans,

gulp-sourcemaps just published its new version 1.9.2.

State Failing tests 🚨
Dependency gulp-sourcemaps
New version 1.9.2
Type devDependency

This version is covered by your current version range and after updating it in your project the build went from success to failure.

As gulp-sourcemaps is “only” a devDependency of this project it might not break production or downstream projects, but “only” your build or test tools – preventing new deploys or publishes.

I recommend you give this issue a high priority. I’m sure you can resolve this 💪

Of course this could just be a false positive, caused by a flaky test suite, or third parties that are currently broken or unavailable, but that would be another problem I’d recommend working on.

Do you have any ideas how I could improve these pull requests? Did I report anything you think isn’t right?
Are you unsure about how things are supposed to work?

There is a collection of frequently asked questions and while I’m just a bot, there is a group of people who are happy to teach me new things. Let them know.

Good luck with your project ✨

You rock!

🌴


The new version differs by 58 commits .

  • b04a217 bump
  • 1f3cfb8 preExisting still rewrite urls but do not add extra newline
  • a83c5fa breaking up init for more readability
  • 335aa4f .gitignore debug folder
  • b1253aa two tasks joining sourcemaps
  • 43b20fb integration example with preExisting and non existent 'map' files
  • 645bee3 more integration specs with gulp concat
  • da0efa8 bump 1.9.1 for pre-release for 1.X
  • 66e6d2a Merge pull request #255 from realtymaps/dev/nem/1.X/preExistingSourcemapFlag
  • b7f1dbf fix comment
  • 970cb95 test only LTS node
  • 1a437f7 fix for crappy 0.10.X node
  • 6972337 preExisting sourceMapping checks to not modify the original file
  • 25f5778 .gitignore for *.lock for people using yarn
  • 0290827 bumping to 1.9.X for engines def

There are 58 commits in total. See the full diff.


This pull request was created by greenkeeper.io.

Tired of seeing this sponsor message? ⚡ greenkeeper upgrade

@coveralls
Copy link

coveralls commented Jan 8, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling c8e36b3 on greenkeeper-gulp-sourcemaps-1.9.2 into 2b05003 on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling c8e36b3 on greenkeeper-gulp-sourcemaps-1.9.2 into 2b05003 on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants