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(bazel): point to the new material.angular.io workspace #30209

Closed
wants to merge 2 commits into from

Conversation

alexeagle
Copy link
Contributor

@alexeagle alexeagle commented Dec 18, 2024

Bazel needs to be told about the nested workspace, to avoid descending into that folder.

Followup for #30208

Bazel needs to be told about the nested workspace, to avoid descending into that folder.
@alexeagle alexeagle requested a review from a team as a code owner December 18, 2024 18:07
@alexeagle alexeagle requested review from mmalerba and andrewseguin and removed request for a team December 18, 2024 18:07
@alexeagle alexeagle changed the title chore(bazel): point to the new material.angular.io workspace build(bazel): point to the new material.angular.io workspace Dec 18, 2024
@andrewseguin andrewseguin added the action: merge The PR is ready for merge by the caretaker label Dec 18, 2024
@alexeagle
Copy link
Contributor Author

I'm not able to amend the commit message in the GitHub UI and the husky pre-commit hook is breaking my local git since it expects yarn on the PATH. Maybe this can merge on red.

@alexeagle
Copy link
Contributor Author

Replaced with #30210 to get it green

@alexeagle alexeagle closed this Dec 18, 2024
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants