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

refactor: remove packages/angular #3045

Merged
merged 1 commit into from
Nov 3, 2021
Merged

refactor: remove packages/angular #3045

merged 1 commit into from
Nov 3, 2021

Conversation

alexeagle
Copy link
Collaborator

It is moving to just-jeb/angular-builders#1048

We announced the deprecation in the README of this package for 4.x

It is moving to just-jeb/angular-builders#1048

We announced the deprecation in the README of this package for 4.x
@alexeagle alexeagle requested a review from jbedard November 1, 2021 21:02
@jbedard
Copy link
Collaborator

jbedard commented Nov 1, 2021

Should we add a link to that project? Or just as part of the 5.0 changelog?

@alexeagle
Copy link
Collaborator Author

@jbedard where are you suggesting that link would go? it's already on https://www.npmjs.com/package/@bazel/angular in the README for anyone prior to the next release to see

@jbedard
Copy link
Collaborator

jbedard commented Nov 3, 2021

I guess I was thinking in the changelog? Although I don't see any other reference to this project in the changelog so 🤷

@alexeagle
Copy link
Collaborator Author

Looking for an approve from you so I can merge

Will def be in the 5.0 release notes

@alexeagle alexeagle merged commit 8b4733b into 5.x Nov 3, 2021
@alexeagle alexeagle deleted the rm_angular_builder branch August 10, 2022 21:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants