Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upAdd eslint to project #89
Comments
ruebot
self-assigned this
Mar 29, 2018
This comment has been minimized.
This comment has been minimized.
I'm not going to add this completely since we'll have more system dependencies for testing - |
This comment has been minimized.
This comment has been minimized.
Actually, it'll just skip over it if you don't do |
This comment has been minimized.
This comment has been minimized.
Might need to open up a separate issue for this, and see if we can address it. For now, I'll just ignore the file. |
This comment has been minimized.
This comment has been minimized.
I'll clean this up and then put a PR in, and have @greebie and @ianmilligan1 test it since I'll be working with
|
ruebot
added a commit
that referenced
this issue
Apr 8, 2018
ianmilligan1
closed this
in
d15b88b
Apr 9, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
ruebot commentedMar 29, 2018
We use this on warcbase to lint js files. We should do it here as well since we're adding it with #85.
I'll take care of this after #85 is merged.