fix(compiler-cli): don't report emit diagnostics when --noEmitOnError…

[alexeagle]

… is off

PR Checklist

Please check if your PR fulfills the following requirements:

• The commit message follows our guidelines: https://github.com/angular/angular/blob/master/CONTRIBUTING.md#commit
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] angular.io application / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

[mary-poppins]

You can preview 1030574 at https://pr20063-1030574.ngbuilds.io/.

[tbosch]

Could you add a test in program_spec.ts? There already is one for noEmitOnError=true, you can copy and modify it for noEmitOnError=false.

[alexeagle]

done

[mary-poppins]

You can preview 5d7dede at https://pr20063-5d7dede.ngbuilds.io/.

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