Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Break the build on lint errors? #31

Closed
jflasher opened this issue Aug 6, 2015 · 2 comments
Closed

Break the build on lint errors? #31

jflasher opened this issue Aug 6, 2015 · 2 comments

Comments

@jflasher
Copy link
Contributor

jflasher commented Aug 6, 2015

@anandthakker mentioned this in #30 and while I was previously against it, having done it on a few projects now, I think I'm more into it. My worries about not being able to deploy are outweighed by my love of cleanly linted files.

As for tooling, we're using semistandard and I know it has tooling for ST and Atom and I imagine vim will work with anything via the command line?
@anandthakker
Copy link
Contributor

I imagine vim will work with anything via the command line?

Yep, pretty much -- I recommend using Syntastic

@jflasher
Copy link
Contributor Author

Closing this as we've generally implemented breaking on staging and non-breaking on production.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jflasher @anandthakker