Build: add hound config #13

Closed
joeybaker wants to merge 1 commits from add-hound into master
joeybaker commented 2016-07-22 17:03:24 +00:00 (Migrated from github.com)

So that we can run eslint automatically

So that we can run eslint automatically
joeybaker commented 2016-07-22 17:10:40 +00:00 (Migrated from github.com)

@mdibaiee If you think this is a good idea, you'll have to enable this at: https://houndci.com/repos

@mdibaiee If you think this is a good idea, you'll have to enable this at: https://houndci.com/repos
mdibaiee commented 2016-07-22 17:14:38 +00:00 (Migrated from github.com)

@joeybaker: We already have ghooks to prevent commiting without proper style, I don't really like the comment thing.

@joeybaker: We already have ghooks to prevent commiting without proper style, I don't really like the comment thing.
joeybaker commented 2016-07-22 17:14:53 +00:00 (Migrated from github.com)

Fair enough.

Fair enough.
joeybaker commented 2016-07-22 17:15:33 +00:00 (Migrated from github.com)

Just noting: it will fail the build in addition to adding comments.

Just noting: it will fail the build in addition to adding comments.

Pull request closed

Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: thereadme/hapi-sequelize-crud#13
No description provided.