refactor(serve): remove unused option baseURL

[mham]

Fixes #3531

[hansl]

This doesn't seem like it's fixing the issue you're pointing at. And also baseUrl is actually used.

I think the bug is subtler than this; we don't reuse commandOptions when assigning it, so it gets overwritten.

In any case, I'd like to have an e2e test that creates an invalid config/environment.js file (just throw an error) and tries to build and serve. This way at least we're sure that the environment isn't taken into account anymore.

[filipesilva]

Superseded by #4057, which is the real fix.

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}