-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
added support for shippable #462
Conversation
You do need a couple of installed libraries to run shields: phantomjs and cairo (see here). It would be preferable to avoid SVG scraping. Do you know if there are plans to support this in Shippable's standard API? Your PR as it stands triggers an Internal Server Error in shippable's server. |
cache(function(data, match, sendBadge, request) { | ||
var project = match[2]; // eg, 54d119db5ab6cc13528ab183 | ||
var branch = match[3]; | ||
var format = match[4]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2, 3, 4? I think 1, 2, 3.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, sorry..
Thanks for contributing anywho! |
@espadrine thing is: for some reason I can't install cairo here.. got different errors every time... Don't know about the shippable api.. Thanks! |
@espadrine @caarlos0 it seems like this service no longer works, it requires adding a whitespace replace to
Sending in a PR for the same |
@mahirk yeah, maybe they changed the API... |
@@ -312,6 +312,41 @@ cache(function(data, match, sendBadge, request) { | |||
}); | |||
})); | |||
|
|||
// Shippable integration | |||
camp.route(/^\/shippable?\/([^\/]+)(?:\/(.+))?\.(svg|png|gif|jpg|json)$/, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The first ?
seems unnecessary.
I wasn't able to test it in my machine though (problems with node-gyp)...
Anyways, I think it should work that way..