An in-range update of pre-commit is breaking the build 🚨

greenkeeper[bot] commented 7 years ago

Version 1.1.3 of pre-commit just got published.

Branch	Build failing 🚨
Dependency	pre-commit
Current Version	1.1.2
Type	devDependency

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

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 :muscle:

Status Details

- ❌ **coverage/coveralls** Coverage pending from Coveralls.io [Details](https://coveralls.io/builds/9011538) - ❌ **continuous-integration/travis-ci/push** The Travis CI build failed [Details](https://travis-ci.org/unshiftio/url-parse/builds/179157395)

Commits

The new version differs by 11 commits .

0f0833a [dist] 1.1.3
fa6119d Merge pull request #57 from blacksonic/master
2abd42b Merge pull request #64 from kevinoid/relative-link
1a04430 Make pre-commit hook symlink relative
2ca2fb9 Merge pull request #60 from saiichihashimoto/patch-1
b6e63d7 Update README.md
4510f45 [fix] add .bashrc to supported home files
3f83f72 Merge pull request #54 from fhemberger/fix/dependencies
683221f Merge pull request #53 from fhemberger/fix/travis
53ec6e9 Update dependencies
eb7ac24 [travis] Add Node@4.x, use new container infrastructure

See the full diff.

Not sure how things should work exactly?

There is a collection of [frequently asked questions](https://greenkeeper.io/faq.html) and of course you may always [ask my humans](https://github.com/greenkeeperio/greenkeeper/issues/new).

Your Greenkeeper Bot :palm_tree:

greenkeeper[bot] commented 7 years ago

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

3rd-Eden commented 7 years ago

@lpinca is there any way we can resolve those localtunnel.me errors?

On Nov 27, 2016, at 7:43 AM, greenkeeper[bot] notifications@github.com wrote:

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

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

lpinca commented 7 years ago

We have few options:

self host a localtunnel server which requires a server and a domain
use ngrok for $5/mo or try to reduce the concurrency to stay in the limits imposed by the free plan
check other alternatives like https://meetfinch.com. This one looked pretty good and writing an adapter for zuul seems easy as there is a client for Node.js but again the free plan is too limited as it only allows 60 minutes monthly usage.

Trying to use ngrok limiting the currency is probably the only way if we don't want to spend money.

lpinca commented 7 years ago

I tried with:

Sauce Connect instead of ngrok or localtunnel. It doesn't work with Safari, iOS things and Microsoft Edge.
ngrok limiting the cuncurrency. It doesn't work as the 40 connections / minute limit is hit even when concurrency is set to 1.

I can only think of using a selfhosted localtunnel server. Perhaps GoDaddy can sponsor that? We can also try to replace zuul with something else but I'm not aware of any good alternative.

3rd-Eden commented 7 years ago

@lpinca I think that going with a self hosted localtunnel would be the best solution here. I still have a useless domain that I don't use nope-dont.care and I could probalby just spin up something at DigitialOcean.

lpinca commented 7 years ago

@3rd-Eden sounds good!

greenkeeper[bot] commented 7 years ago

Version 1.2.1 just got published.

Your tests are passing again with this version. Explicitly upgrade to this version 🚀

Commits

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.

greenkeeper[bot] commented 7 years ago

Version 1.2.2 just got published.

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

Commits

The new version differs by 12 commits .

bf393ad [dist] 1.2.2
675560c Use string instead of int 777 in fs.chmodSync so it gets correctly interpreted as octal (#90)
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.

unshiftio / url-parse