Enabled ESLint and JSCS in Hound #19482

Closed
wants to merge 1 commit into from

3 participants

@ameya-pandilwar

Fix for #19114
Closes #19299

@ameya-pandilwar ameya-pandilwar Enabled ESLint and JSCS in Hound
f24a018
@XhmikosR
Bootstrap member
git rebase -i origin/v4-dev

Assuming you have set up upstream right.

@XhmikosR
Bootstrap member

@cvrebert: If we have the checks in hound, shouldn't we stop checking those in Travis? It doesn't take long, sure, but it's just duplicate tasks.

@cvrebert
Bootstrap member

Meh. Hound's support for ESLint & JSCS is still in beta. And what if Hound got shut down or became flaky?
(My feelings aren't strong, but anyway that's my 2¢.)

@cvrebert cvrebert added this to the v4.0.0-alpha.3 milestone Mar 15, 2016
@cvrebert cvrebert added a commit that referenced this pull request Mar 15, 2016
@ameya-pandilwar ameya-pandilwar Enabled ESLint and JSCS in Hound
Fixes #19114
Closes #19482, #19299
122d9e5
@cvrebert
Bootstrap member

@ameya-pandilwar Merged as 122d9e5. Thanks!
Also, for future reference, utilizing https://help.github.com/articles/closing-issues-via-commit-messages/ in commit messages is preferred.

@cvrebert cvrebert closed this Mar 15, 2016
@cvrebert cvrebert added a commit that referenced this pull request Mar 15, 2016
@cvrebert cvrebert Port #19482 to v3
[ci skip]
55f6efb
@ameya-pandilwar ameya-pandilwar deleted the ameya-pandilwar:fix-enable-eslint-jscs branch Mar 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment