Skip to content

Conversation

@noordean
Copy link
Owner

@noordean noordean commented Apr 4, 2018

No description provided.

@@ -0,0 +1,14 @@
import chai from 'chai';

const should = chai.should();

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'should' is assigned a value but never used no-unused-vars

@@ -0,0 +1,14 @@
import chai from 'chai';

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unable to resolve path to module 'chai' import/no-unresolved
Missing file extension for "chai" import/extensions

Copy link
Collaborator

@phemmz phemmz Apr 5, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can try to add this to .eslintrc to fix this:

      "settings": {
          "import/resolver": {
              "node": {
                  "extensions": [".js",".jsx"]
              }
          }
     }

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've added this and it's still not working. Help me check the .eslintrc to see if I've added it correctly.

@noordean noordean requested a review from phemmz April 4, 2018 10:19
- add .hound.yml and .travis.yml files
- install mochas, nyc and chai
- write a sample test
- add npm test to package.json

[Finishes #156465076]
@noordean noordean force-pushed the ch-integrate-travis-coveralls-houndci-156465076 branch from acd7a29 to 490b012 Compare April 5, 2018 15:25
@@ -0,0 +1,14 @@
import chai from 'chai';

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unable to resolve path to module 'chai' import/no-unresolved
Missing file extension for "chai" import/extensions

@phemmz phemmz merged commit fc7eacc into develop Apr 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants