Skip to content

docs: update long-description#32768

Open
SkyZeroZx wants to merge 1 commit intoangular:mainfrom
SkyZeroZx:docs/update-docs-cli
Open

docs: update long-description#32768
SkyZeroZx wants to merge 1 commit intoangular:mainfrom
SkyZeroZx:docs/update-docs-cli

Conversation

@SkyZeroZx
Copy link

What is the current behavior?

image

What is the new behavior?

Avoid using absolute links in angular.dev since this is an internal route.

image

@angular-robot angular-robot bot added the area: docs Related to the documentation label Mar 14, 2026
Copy link

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

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 updates the documentation for the ng update command. The changes involve updating an example version from 10 to 21 and converting an absolute URL for the Angular Update Guide to a relative path. While the link change is a good improvement, the hardcoded version number in the example could quickly become outdated. I've provided a suggestion to rephrase the example to make it more robust and clearer for users.

Copy link
Collaborator

@alan-agius4 alan-agius4 left a comment

Choose a reason for hiding this comment

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

LGTM thanks

@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 and removed area: docs Related to the documentation labels Mar 16, 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 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