-
-
Notifications
You must be signed in to change notification settings - Fork 152
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.
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
fix(textlint-formatter): run all tests #403
Conversation
Correct mocha test glob pattern to run all tests. Closes textlint#402.
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.
Does the tests pass in your local?
Probably, color is disabled in CI env and pass tests.
@@ -16,7 +16,7 @@ | |||
"bin/" | |||
], | |||
"scripts": { | |||
"test": "mocha test/*.js" | |||
"test": "mocha test/**/*.js" |
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.
Can you try --no-color option?
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.
Run test from top directory without --no-color
have passed successfully.
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.
Should I add --no-color
option to the package.json
, or leave it as it is (at this moment, and fix test cases later)?
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.
Please add --no-color
option.
master branch should be always passed tests in any environment.
Disable colors in order to get consistent test result.
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.
LGTM. Thanks!
Thank you! |
Correct mocha test glob pattern to run all tests. Closes #402.