Skip to content

build: pretty-print adev CDK docs manifests #31043

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 7, 2025

Conversation

devversion
Copy link
Member

This should make it easier to diff the API changes when PRs are auto-opened in the framework repository.

This should make it easier to diff the API changes when PRs are
auto-opened in the framework repository.
@devversion devversion requested a review from a team as a code owner May 7, 2025 11:44
@devversion devversion requested a review from mmalerba May 7, 2025 11:44
@angular-robot angular-robot bot added the area: build & ci Related the build and CI infrastructure of the project label May 7, 2025
Copy link
Contributor

@mmalerba mmalerba left a comment

Choose a reason for hiding this comment

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

LGTM, FYI you may want to make the same change here: https://github.com/angular/angular/blob/main/adev/shared-docs/pipeline/api-gen/extraction/index.ts#L95 to make eventual sharing of the script easier

@mmalerba mmalerba added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels May 7, 2025
@mmalerba mmalerba merged commit eb06128 into angular:main May 7, 2025
29 checks passed
mmalerba pushed a commit that referenced this pull request May 7, 2025
This should make it easier to diff the API changes when PRs are
auto-opened in the framework repository.

(cherry picked from commit eb06128)
mmalerba pushed a commit that referenced this pull request May 7, 2025
This should make it easier to diff the API changes when PRs are
auto-opened in the framework repository.

(cherry picked from commit eb06128)
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: build & ci Related the build and CI infrastructure of the project 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