Skip to content

revert(@angular/cli): import markdown files directly for command descriptions - #34109

Merged
alan-agius4 merged 1 commit into
angular:22.2.xfrom
alan-agius4:revert-3d7f081-22.2.x
Sep 17, 2026
Merged

alan-agius4 merged 1 commit into
angular:22.2.xfrom
alan-agius4:revert-3d7f081-22.2.x

Conversation

@alan-agius4

Copy link
Copy Markdown
Collaborator

Reverts commit 3d7f081 in 22.2.x.

@alan-agius4 alan-agius4 added target: rc This PR is targeted for the next release-candidate action: merge The PR is ready for merge by the caretaker labels Sep 17, 2026

@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 refactors how long descriptions (Markdown files) are loaded for Angular CLI commands. It removes the custom markdown loader hook and direct imports of .md files, replacing them with a longDescriptionPath property on command modules to dynamically read the files at runtime. The review feedback highlights an issue with the relative path calculation in command-module.ts, which omits the @angular/cli/ prefix and resolves to angular/cli/src/commands/... instead, potentially breaking documentation generation tools.

Comment thread packages/angular/cli/src/command-builder/command-module.ts
@alan-agius4
alan-agius4 requested a review from clydin September 17, 2026 16:31
@alan-agius4
alan-agius4 merged commit 9c6037a into angular:22.2.x Sep 17, 2026
36 checks passed
@alan-agius4

Copy link
Copy Markdown
Collaborator Author

This PR was merged into the repository. The changes were merged into the following branches:

@alan-agius4
alan-agius4 deleted the revert-3d7f081-22.2.x branch September 17, 2026 16:48
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: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants