-
Notifications
You must be signed in to change notification settings - Fork 6
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
Deal with Github repo redirects #36
Conversation
@@ -78,8 +84,7 @@ def handle_update_error(e, pod) | |||
when /404 Not Found/ | |||
not_found(pod) | |||
when /403 API rate limit exceeded/ | |||
headers = ::Github::Response::Headers.new response_headers: e.http_headers | |||
|
|||
headers = ::Github::Response::Header.new response_headers: e.http_headers |
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.
There is no Headers
class only Header, this was being masked by the raise
in updater.rb
@@ -41,6 +41,11 @@ begin | |||
exec 'bundle exec kicker -c' | |||
end | |||
|
|||
desc 'Update metrics for a pod' | |||
task :update, [:name] do |t, args| | |||
exec "bundle exec ruby updater.rb #{ args[:name] }" |
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.
Missing an end
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.
thanks 👯
doing it 🍏 |
@@ -22,6 +22,7 @@ Metrics is an API to view metrics for Pods. | |||
+ Attributes | |||
+ github (GitHub Metrics) | |||
+ cocoadocs (CocoaDoc Metrics) | |||
+ stats (CocoaPods Stats Metrics) |
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.
http://metrics.cocoapods.org/api/v1/status
has github and cocoadocs, should it also have stats now?
We were getting crashes that stopped the github metrics grabbing caused by redirects. It was difficult to run just one metric, so I added a rake task for it.