Skip to content
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

build: migrate tests infrastructure to ts_project #29242

Merged
merged 1 commit into from
Jan 6, 2025

Conversation

clydin
Copy link
Member

@clydin clydin commented Jan 2, 2025

The miscellaneous test infrastructure targets within tests/ have been migrated to the rules_js ts_project rule.

The miscellaneous test infrastructure targets within `tests/` have been migrated to the `rules_js` ts_project rule.
@clydin clydin added the target: patch This PR is targeted for the next patch release label Jan 2, 2025
@angular-robot angular-robot bot added the area: build & ci Related the build and CI infrastructure of the project label Jan 2, 2025
@clydin clydin requested a review from devversion January 2, 2025 15:49
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Jan 2, 2025
@clydin clydin requested a review from josephperrott January 3, 2025 17:52
Copy link
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

@josephperrott josephperrott removed the request for review from devversion January 6, 2025 18:08
@clydin clydin added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jan 6, 2025
@clydin clydin merged commit 7b15a15 into angular:main Jan 6, 2025
34 checks passed
@clydin
Copy link
Member Author

clydin commented Jan 6, 2025

The changes were merged into the following branches: main, 19.0.x

@clydin clydin deleted the rulesjs/migrate-tests branch January 6, 2025 18:11
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