Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

Refactor to remove phantomjs/main.js #63

Open
pconerly opened this issue Apr 10, 2013 · 1 comment
Open

Refactor to remove phantomjs/main.js #63

pconerly opened this issue Apr 10, 2013 · 1 comment

Comments

@pconerly
Copy link

Hi,

I don't have a specific solution but I think that the current organization of grunt-mocha leaves something to be desired.

Should there be a separate copy of phantomjs/main.js within grunt-mocha? I know you made a modification to it, but I feel like it would be better to see if we could just pass in the required grunt-mocha options into the existing grunt-lib-phantomjs/phantomjs/main.js file.

Thoughts?

@kmiyashiro
Copy link
Owner

I agree!

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

No branches or pull requests

2 participants