Skip to content
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 --watch not finding any files to execute; closes #3748 #3755

Merged
merged 1 commit into from
Feb 21, 2019

Conversation

boneskull
Copy link
Contributor

...missing parameter.

@boneskull boneskull added type: bug a defect, confirmed by a maintainer semver-patch implementation requires increase of "patch" version number; "bug fixes" labels Feb 20, 2019
@boneskull boneskull self-assigned this Feb 20, 2019
@boneskull boneskull added this to the next milestone Feb 20, 2019
@boneskull
Copy link
Contributor Author

appveyor is killing me today

@boneskull boneskull merged commit b7cfceb into master Feb 21, 2019
@boneskull boneskull deleted the boneskull/issue/3748 branch February 21, 2019 05:23
@plroebuck
Copy link
Contributor

plroebuck commented Feb 21, 2019

BTW...

$ DEBUG=* /var/tmp/mocha-ya/bin/_mocha --require coffee-script/register --extension="coffee"
  mocha:cli:cli entered main with raw args [ '--require',
  'coffee-script/register',
  '--extension=coffee' ] +0ms
  mocha:cli:options no config found in /var/tmp/sample-node-typescript-module/package.json +0ms
  mocha:cli:options No mocha.opts found at ./test/mocha.opts +48ms
  mocha:cli:run:helpers loaded require "coffee-script/register" +0ms
  mocha:cli:run post-yargs config { _: [],
  diff: true,
  require: [ 'coffee-script/register' ],
  r: [ 'coffee-script/register' ],
  extension: [ 'coffee', 'js' ],
  'watch-extensions': [ 'coffee', 'js' ],       // <= :NOTE: camelCase version is different...
  watchExtensions: [ 'coffee' ],
  opts: false,
  reporter: 'spec',
  R: 'spec',
  slow: 75,
  s: 75,
  timeout: 2000,
  t: 2000,
  timeouts: 2000,
  ui: 'bdd',
  u: 'bdd',
  spec: [ 'test/' ],
  config: false,
  package: false,
  '$0': 'mocha' } +0ms
...

@boneskull
Copy link
Contributor Author

camelcase args are thrown out at some point in there before we get to _mocha

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver-patch implementation requires increase of "patch" version number; "bug fixes" type: bug a defect, confirmed by a maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants