-
Notifications
You must be signed in to change notification settings - Fork 26
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
Adds health checking to the board loading sequence #27
Conversation
@@ -0,0 +1,50 @@ | |||
module HealthChecker | |||
module HealthChecks | |||
class GitHubRepoWebhooks |
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.
Suffix the check classes with Check
so they are easy to search for
|
||
#Supports carrying over DSL attributes to the instantiated | ||
#parent class | ||
def initialize |
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.
initialize logic added by modules seems dangerous.
I can think of few libraries that do it but it's always been weird to track it down.
It's just my gut, I'm willing to be wrong
… initializing instance vars and calling super
…weaks a payload mispell
…in another PR should be used on trials)
… autoload problems
…es if you are logged in, thats the concern of the health check
Needs UI work |
Conflicts: Gemfile.lock config/routes.rb
Conflicts: config/routes.rb ember-app/app/routes/application.js
|
||
var Health = Settings.extend({ | ||
repo: {}, | ||
storageKey: Ember.computed('repo.data.repo.full_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.
Cool! For troubleshooting convenience, we should may add a refresh button that refetches all the checks ?
makes health checks linked repo aware
HealthChecking lib for the back-end, comprised of DSL style 'health checks' that respond to
perform