thereadme/hapi-sequelize-crud
1
0
Fork 0
Code Issues 4 Pull Requests 1 Projects Releases Wiki Activity

Build: add hound config #13

Closed
joeybaker wants to merge 1 commits from add-hound into master
pull from: add-hound
merge into: :master
Conversation 4 Commits 1 Files Changed 1
+6
joeybaker commented 2016-07-22 17:03:24 +00:00 (Migrated from github.com)
Copy Link
Copy Source

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)
Copy Link
Copy Source

@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)
Copy Link
Copy Source

@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)
Copy Link
Copy Source

Fair enough.

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

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

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
thereadme
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: thereadme/hapi-sequelize-crud#13