fix(command-deploy): remove random spinner

[erezrokah]

- Summary

Fixes #2086
Fixes #2557

After adding some debug logs, specifically logging the error here:

cli/src/utils/telemetry/tracked-command.js

Line 38 in fa7cd00

async finally(error) {

I was able to reproduce the issue and get the stacktrace for it. The offending code is here:
https://github.com/sindresorhus/ora/blob/c7d6dba4a723cfa640e6adb716cb6b5dad2c22d2/index.js#L249
frames appears to be undefined.

The underlying issue is that the following line:

cli/src/commands/deploy.js

Line 6 in fa7cd00

const cliSpinnerNames = Object.keys(require('cli-spinners'))

adds 'default' to the cliSpinnerNames array and that is not a valid spinner name.
Spinner names are validated here:
https://github.com/sindresorhus/ora/blob/c7d6dba4a723cfa640e6adb716cb6b5dad2c22d2/index.js#L171

but that code suffers from a similar bug of allowing 'default'.

I removed using a random spinner since it feels confusing anyway, however if we want to keep it we can simply use spinner: 'random'.

- Test plan

Existing tests

- A picture of a cute animal (not mandatory but encouraged)
🐒

[eduardoboucas]

Wow! 😮

[erezrokah]

Also sindresorhus/ora#175