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

pre-commit@1.2.1 breaks build 🚨 #29

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

Conversation

greenkeeperio-bot
Copy link
Contributor

Hello lovely humans,

pre-commit just published its new version 1.2.1.

State Failing tests 🚨
Dependency pre-commit
New version 1.2.1
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 pre-commit 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 10 commits .

  • 7d6feb1 [dist] 1.2.1
  • 3ecafe0 [fix] chmod 0777 the newly created file; Fixes #88
  • 1689e3e [dist] 1.2.0 - Better windows support
  • a9c9732 solves the windows issue (with symlinks) (#84)
  • 4167676 chore(package): update mocha to version 3.2.0 (#86)
  • 3cf4cf7 chore(package): update dependencies (#85)
  • 7154dee Update dep to remove warn about cross-spaw (#76)
  • ead9484 cross-spawn@4 (#77)
  • fec48cb [ci] Remove NPM hack and old npm
  • 18da668 [deps] Fix npmgate

See the full diff.


This pull request was created by greenkeeper.io.

Tired of seeing this sponsor message? ⚡ greenkeeper upgrade

@cchan cchan force-pushed the greenkeeper-pre-commit-1.2.1 branch from abc696a to ff58a7d Compare December 25, 2016 22:20
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.

2 participants