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

--cover #12

Open
yields opened this issue Jul 7, 2014 · 5 comments
Open

--cover #12

yields opened this issue Jul 7, 2014 · 5 comments

Comments

@yields
Copy link
Contributor

yields commented Jul 7, 2014

using duo api.

  1. .use(cover)
  2. if it's an entry instrument it
  3. if it's a local dep of an entry like ./local.js instrument it.
  4. profit
@stephenmathieson
Copy link
Contributor

yes please

On Jul 7, 2014, at 2:53 PM, amir abu shareb [email protected] wrote:

using duo api.

.use(cover)
if it's an entry instrument it
if it's a local deep of an entry like ./local.js instrument it.
profit

Reply to this email directly or view it on GitHub.

@yields
Copy link
Contributor Author

yields commented Jul 9, 2014

i don't think we should instrument in duo-test, it defeats the purpose of it being simple to configure.

instead i propose we add --cover <entry.js> to duo itself, duo will then instrument using a utility and expose a global like __cover__ or something that duo test can utilize to show stats / render.

any thoughts / feedback are welcome!

@matthewmueller

@matthewmueller
Copy link
Contributor

any chance it could be a separate script and run under duo cover <entry>?

@yields
Copy link
Contributor Author

yields commented Jul 9, 2014

don't think so :( there's the case were we do stuff like duo test.js > build.js and run the tests,
so instead we would do duo test.js --cover index.js > build.js.

@stephenmathieson
Copy link
Contributor

any updates here? this would be super useful right now

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

No branches or pull requests

3 participants