Setup ESlint for project; resolves #89. #97
Conversation
- Add ESlint config - Update gitignore - Update TravisCI config - Add Rake task - Clean-up JS files - Ignore some JS files
Oh yeah, TravisCI runs the ESlinting now too, so that should be good too. |
Codecov Report
@@ Coverage Diff @@
## master #97 +/- ##
=======================================
Coverage 87.02% 87.02%
=======================================
Files 26 26
Lines 239 239
=======================================
Hits 208 208
Misses 31 31 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested – both successfully with everything, and also successfully caught that files were missing when they were removed.
{ | ||
"extends": "airbnb-base/legacy", | ||
"globals": { | ||
"jQuery": true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add sigma
as a global?
lgtm. Only comment is the optional add of |
I've tested locally by moving the
gexf
file, and also removing all the nodes and edges to simulate an empty graph so recreate the two conditionals. Everything looks fine, but just want to make sure things are good not on my laptop.@greebie and @ianmilligan1 would you mind doing a smoke test on your end?