feat(vite): add v8 coverage support to vitest generator

[mpgregson]

Current Behavior

Vitest generator has no option for v8 coverage and defaults to c8

Expected Behavior

Vitest generator should allow (and default to) v8 coverage

Related Issue(s)

Fixes #17886

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
nx-dev	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 27, 2023 6:41am

[mpgregson]

I've been unable to verify this is working as expected as I received errors when trying to create a new workspace using the locally published package (errors seemed unrelated to my changes).

Could someone help me test/verify that it's working as expected?

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 05184d4. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

---

✅ Successfully ran 8 targets

• linux nx affected --target=e2e --base=f113bea262e5600d49953d124be00af9a0741cc9 --head=05184d4259791e3e0aae31751d9eeca7d99387f6 --parallel=1
• linux nx affected --target=test --base=f113bea262e5600d49953d124be00af9a0741cc9 --head=05184d4259791e3e0aae31751d9eeca7d99387f6 --parallel=1
• linux nx affected --target=build --base=f113bea262e5600d49953d124be00af9a0741cc9 --head=05184d4259791e3e0aae31751d9eeca7d99387f6 --parallel=3
• linux nx run-many -t check-imports check-commit check-lock-files check-codeowners documentation --parallel=1 --no-dte
• linux nx affected --target=lint --base=f113bea262e5600d49953d124be00af9a0741cc9 --head=05184d4259791e3e0aae31751d9eeca7d99387f6 --parallel=3
• nx-cloud record -- nx format:check --base=f113bea262e5600d49953d124be00af9a0741cc9 --head=05184d4259791e3e0aae31751d9eeca7d99387f6
• linux nx documentation --no-dte
• macos nx affected -t e2e-macos --parallel=1 --base=f113bea262e5600d49953d124be00af9a0741cc9 --head=05184d4259791e3e0aae31751d9eeca7d99387f6

---

Sent with 💌 from NxCloud.

[barbados-clemens]

Just a couple changes and you'll want to rebase with master.

[mpgregson]

Just a couple changes and you'll want to rebase with master.

I think I've addressed the comments correctly, and rebased. Let me know if any further changes are required 👍

[github-actions]

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.