fix(vite): a couple of minor fix

[vicb]

• close the dev server before process exists,
• use server.resolvedUrls for the base url.

Note that server.resolvedUrls is what is used by server.printUrls(); that the executor calls on l.71

/cc @mandarini

[vercel]

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

1 Ignored Deployment

Name	Status	Preview	Updated
nx-dev	⬜️ Ignored (Inspect)		Jan 12, 2023 at 5:48PM (UTC)

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit d76f5c5. 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 5 targets

• nx affected --target=e2e --base=ba3fd62b7e4835cf7c53fad13a4b630e3127aa86 --head=d76f5c5210a2cbf2842222340a47e3b023356baa --exclude=e2e-storybook,e2e-storybook-angular,e2e-react-native,e2e-detox --parallel=1
• nx affected -t test lint --base=ba3fd62b7e4835cf7c53fad13a4b630e3127aa86 --head=d76f5c5210a2cbf2842222340a47e3b023356baa --parallel=1
• nx affected --target=build --base=ba3fd62b7e4835cf7c53fad13a4b630e3127aa86 --head=d76f5c5210a2cbf2842222340a47e3b023356baa --parallel=3
• nx run-many -t check-imports check-commit check-lock-files root-lint depcheck documentation --parallel=1 --no-dte
• nx-cloud record -- nx format:check --base=ba3fd62b7e4835cf7c53fad13a4b630e3127aa86 --head=d76f5c5210a2cbf2842222340a47e3b023356baa

---

Sent with 💌 from NxCloud.

[mandarini]

@vicb thanks for all your work on @nrwl/vite. Do you mind joining the nx-community slack and pinging me there? I'd like to chat about your experience with @nrwl/vite and how you use it, it could be helpful!

If you don't want, that's ok! :)

[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.