Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Jul 29, 2019. It is now read-only.

Conversation

@mojoaxel
Copy link
Member

@mojoaxel mojoaxel commented Dec 5, 2016

Code-Climate provides a nice opportunity to get a feeling for eslint and csslint status.
I would like to integrate linting into the build process but before that we should get the major issues out of the way.
Rules that make no sense for use should be disabled.

@mojoaxel mojoaxel force-pushed the codeClimate branch 2 times, most recently from 5eae95e to dbb3f14 Compare December 5, 2016 10:18
@yotamberk
Copy link
Member

So how do we use this? I'm not sure how to check out this PR

@mojoaxel
Copy link
Member Author

mojoaxel commented Dec 6, 2016

@yotamberk Sorry my mistake of not being clear. You can go to the codeclimate website and see issues found by different lint-tools.
With this PR I activated some more test-tools but disabled some "useless" test.

@yotamberk yotamberk merged commit 15b739d into develop Dec 6, 2016
@mojoaxel
Copy link
Member Author

mojoaxel commented Dec 6, 2016

@yotamberk Thanks!

Now we can start adressing those issues: https://codeclimate.com/github/almende/vis/compare/develop

@yotamberk
Copy link
Member

I'll have to get in to it. Not sure yet the benefits. Looks cool though!

@mojoaxel mojoaxel mentioned this pull request Dec 7, 2016
@mojoaxel mojoaxel modified the milestone: Minor Release v4.18 Dec 28, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants