Skip to content

fix(@angular/build): correct typos and spacing in error and warning messages - #34403

Open
MeAkib wants to merge 1 commit into
angular:mainfrom
MeAkib:fix/message-typos
Open

MeAkib wants to merge 1 commit into
angular:mainfrom
MeAkib:fix/message-typos

Conversation

@MeAkib

@MeAkib MeAkib commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

Several messages printed by the CLI contained typos or ran two sentences together:

  • The application builder named the option "app-shell" in its errors about server and index, but the option is appShell.
  • The emitDecoratorMetadata note printed "by Angularand" and "explictly".
  • The unit-test Karma --filter warning printed "expression.Leading".
  • The provideServerRendering() configuration error printed "provideServerRendering().The".
  • ng config reported "Confguration file cannot be found.".
  • ng update reported "registry package identifer".

Add the missing spaces, fix the spelling and use the actual option name.

…essages

Several messages printed by the CLI contained typos or ran two sentences together:

- The application builder named the option `"app-shell"` in its errors about `server`
  and `index`, but the option is `appShell`.
- The `emitDecoratorMetadata` note printed "by Angularand" and "explictly".
- The unit-test Karma `--filter` warning printed "expression.Leading".
- The `provideServerRendering()` configuration error printed "provideServerRendering().The".
- `ng config` reported "Confguration file cannot be found.".
- `ng update` reported "registry package identifer".

Add the missing spaces, fix the spelling and use the actual option name.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request primarily focuses on fixing typos, correcting string spacing in error and warning messages, and standardizing option names (e.g., changing 'app-shell' to 'appShell' in error messages). Additionally, it reformats the ServerRoute type definition in route-config.ts to a single line. There are no review comments, and I have no feedback to provide.

@alan-agius4 alan-agius4 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alan-agius4 alan-agius4 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@alan-agius4 alan-agius4 added target: patch This PR is targeted for the next patch release action: merge The PR is ready for merge by the caretaker labels Oct 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: @angular/build target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants