Skip to content

Migrate more targets to ts_project and delete @angular/bazel#61843

Closed
devversion wants to merge 7 commits intoangular:mainfrom
devversion:rjs-7
Closed

Migrate more targets to ts_project and delete @angular/bazel#61843
devversion wants to merge 7 commits intoangular:mainfrom
devversion:rjs-7

Conversation

@devversion
Copy link
Copy Markdown
Member

See individual commits

@angular-robot angular-robot bot added the area: build & ci Related the build and CI infrastructure of the project label Jun 3, 2025
@ngbot ngbot bot added this to the Backlog milestone Jun 3, 2025
@devversion devversion force-pushed the rjs-7 branch 3 times, most recently from bb869ac to 07b4818 Compare June 3, 2025 16:54
@devversion devversion requested a review from josephperrott June 3, 2025 16:55
@devversion devversion added the action: review The PR is still awaiting reviews from at least one requested reviewer label Jun 3, 2025
Copy link
Copy Markdown
Member

@josephperrott josephperrott left a comment

Choose a reason for hiding this comment

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

LGTM

@devversion devversion added action: merge The PR is ready for merge by the caretaker target: minor This PR is targeted for the next minor release and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jun 3, 2025
@devversion devversion marked this pull request as ready for review June 3, 2025 19:03
@devversion devversion added action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews and removed action: merge The PR is ready for merge by the caretaker labels Jun 3, 2025
Migrates remaining `ts_project` targets (excluding zone.js) to
`ts_project`.
Migrates `private/testing` to `ng_project`.
Removes the `@angular/bazel` package as we no longer need it. Yay!

It's replaced by `rules_angular`.
Replaces all `ng_package` rule with the new rule from `rules_angular`.
Migrates `packages/localize/schematics` to `ts_project`.
Fixes that the package diff script did not show newly untracked
files in the npm package.
@devversion devversion added action: merge The PR is ready for merge by the caretaker and removed action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews labels Jun 4, 2025
@devversion
Copy link
Copy Markdown
Member Author

This PR was merged into the repository by commit 397416f.

The changes were merged into the following branches: main

@devversion devversion closed this in f779c95 Jun 4, 2025
devversion added a commit that referenced this pull request Jun 4, 2025
Migrates `private/testing` to `ng_project`.

PR Close #61843
devversion added a commit that referenced this pull request Jun 4, 2025
Removes the `@angular/bazel` package as we no longer need it. Yay!

It's replaced by `rules_angular`.

PR Close #61843
devversion added a commit that referenced this pull request Jun 4, 2025
…61843)

Replaces all `ng_package` rule with the new rule from `rules_angular`.

PR Close #61843
devversion added a commit that referenced this pull request Jun 4, 2025
Migrates `packages/localize/schematics` to `ts_project`.

PR Close #61843
devversion added a commit that referenced this pull request Jun 4, 2025
Fixes that the package diff script did not show newly untracked
files in the npm package.

PR Close #61843
@angular-automatic-lock-bot
Copy link
Copy Markdown

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.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Jul 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project PullApprove: disable target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants