Skip to content
This repository has been archived by the owner on Dec 28, 2023. It is now read-only.

feat: support mocha opts #121

Merged
merged 1 commit into from
Nov 7, 2016
Merged

feat: support mocha opts #121

merged 1 commit into from
Nov 7, 2016

Conversation

maksimr
Copy link
Contributor

@maksimr maksimr commented Nov 5, 2016

closes #99

function normalizeOpts (opts) {
opts = [
'require',

Copy link
Member

Choose a reason for hiding this comment

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

why is there an empty line here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@dignifiedquire This line separte options which will pass to mocha in browser from options which use only on the server (like require) and not acceptable in browser

Copy link
Member

Choose a reason for hiding this comment

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

👍

}

var extend = require('util')._extend
return extend(normalizeOpts(require('minimist')(fs.readFileSync(optsPath, 'utf8')
Copy link
Member

Choose a reason for hiding this comment

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

Hmm not sure how I fell about using require inline

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@dignifiedquire I have moved all require to top of the file

@dignifiedquire dignifiedquire merged commit b6af3eb into karma-runner:master Nov 7, 2016
@dignifiedquire
Copy link
Member

Thank you

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants