-
Notifications
You must be signed in to change notification settings - Fork 177
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
Update development branch from master #494
Closed
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Corrected typo in "custom"
Update advanced_browser.md
fix formatting
provide nice defaults for node coverage
found that if you include blanket at the top, it fails without error message (even with the debug flag on) in both FF and Chrome. by comparing to http://blanketjs.org/examples/backbone-koans-qunit/index.html?coverage=true step-by-step, i found out what i was doing wrong. maybe worth mentioning in the instructions in case more people maybe the same noob error :)
mention script-include-order in getting_started_browser.md
- Allow any node v0.10.* versions (previous setting yielded a warning) - Updating xtend dependency (previous version used deprecated object-keys version)
Fix bad anchor
Update xtend dependency and change engine to 0.10.*
…nner - blanketRunner has to inherit from it
New version of mocha requires suiteURL function
Conflicts: docs/getting_started_node.md package.json src/adapters/mocha-blanket.js src/index.js
this is done so 2 files in 2 different folders with the same name are still distinguishable in the output
include relative path from cwd in shortnames
support mocha cli command like this: ``` mocha -r should -r blanket --report travis-cov test/*.test.js ```
Fixing fs.existsSync issues in older versions of node
…ha-require support mocha require multi modules in cli
* grunt.util.async is deprecated. * See http://coffeescript.org/#changelog for details about CoffeeScript 1.7.0.
trailing whitespaces, unused variable declarations, and missing semicolons
falafel is licensed under the MIT license
Update dependencies and devDependencies (Close alex-seville#376)
mocha's [~1.17.1] default HTML reporter is a function object. It needs to be instantiated to work correctly. note that I haven't tested it with other reporters, but there shouldn't be any side-effects.
Added information detailing what script failed to load, and also added the timeout value in the timeout condition's error message. Our project was running into the default 180ms timeout in certain cases and it was unclear what was happening.
Fixed documentation typo
test coffee script in node link was missing.
This is related to mochajs/mocha#1585, yamadapc/mocha-spec-cov-alt#3 and possibly other issues in other repositories. `mocha` spawns a child process `_mocha` in order to support custom node.js flags being passed into it. The newest version of `mocha` (2.2) uses `process.execPath` rather than `process.argv[0]` to spawn the child. This is meant to be able to have better support for things like `io.js`. Because of that, `process.argv[0]` in this line of `blanket` is set to the absolute path to the node.js/io.js/etc. executable, rather than `node`. This fixes that problem by comparing the `basename` of `process.argv[0]` with a list of appropriate executable names.
…ty-with-alternative-runtimes Fix alternative node.js runtimes for mocha
highlight hit lines
Add more helpful information to error message
=BG= Add data-cover-never to documentation
documentation and relative path
Coffeescript loader should obey data-cover-never
…ty-with-alternative-runtimes Fix typo in `src/index.js`
Just a typo fix. `--save-dep` does not exist in options for [`npm install`](https://docs.npmjs.com/cli/install). Almost sure you ment `--save-dev`.
renamed --save-dep to --save-dev
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.