deprecate Bazel builder in @angular/bazel #37190

kyliau · 2020-05-18T21:42:55Z

This PR removes the schematics for Bazel builder from @angular/bazel.
It consists of a few commits:

Add migration doc for existing users and remove Bazel guide from angular.io.
Remove all @angular/bazel schematics code, mainly ng-add and ng-new, and the Bazel builder.
Remove "Building with Bazel" section from angular.io
Remove aio/content/guide/bazel.md from pullapprove
Remove bazel-schematics integration test
Redirect guide/bazel to deprecation doc in Angular repo

PR closes #19058

PR Checklist

Please check if your PR fulfills the following requirements:

The commit message follows our guidelines: https://github.com/angular/angular/blob/master/CONTRIBUTING.md#commit
Tests for the changes have been added (for bug fixes / features)
Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

Bugfix
Feature
Code style update (formatting, local variables)
Refactoring (no functional changes, no api changes)
Build related changes
CI related changes
Documentation content changes
angular.io application / infrastructure changes
Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

Yes
No

Other information

packages/bazel/src/schematics/README.md

IgorMinar

I think that the README.md needs some more work and likely devrel input. Can you please get @mgechev and @StephenFluin to weigh in on the wording? We'll also need @aikidave to look at this before it hits angular.io in June.

In general we don't want to freak people out with this removal. It's not that we don't like Bazel (some of us very much do! 😃), we just found out that our current approach to jam bazel into Angular CLI is not the right one, and we need to pivot. Instead of wrapping Bazel into Angular CLI we found a way how bazel and Angular CLI can coexist for some workflows and for others it's actually better if Bazel wraps Angular CLI.

In order to enable this pivot, we must deprecate the current integration. For people that depend on Angular CLI + Bazel, they can keep on building their apps with the existing rules (ng_module, ng_package, etc), but we don't want new projects to onboard in this way any more.

In the future, we'll most likely remove all of these custom rules and replace them with direct integration from bazel to ngc, etc. But we are nowhere close to that. We'll revisit how this stuff should work once we remove View Engine from our code base and Ivy is the only compilation pipeline out there.

packages/bazel/src/schematics/README.md

aio/content/guide/bazel.md

packages/bazel/src/schematics/README.md

) This commit removes the fields for ng-add, schematics and builder from package.json of `@angular/bazel`. PR Close #37190

This commit improves some wording in the deprecation doc for Bazel builder and schematics in `@angular/bazel` and fixes the formatting. PR Close #37190

This commit adds a link to the Bazel prototype for orchestrating multiple CLI architects and also adds a link to the #angular channel in the Bazel Slack workspace. PR Close #37190

…37190) This commit adds Bazel builder and schematics to the global list of deprecations in Angular. A link to the migration doc is added. PR Close #37190

This commit adds a deprecation doc for Bazel builder in `@angular/bazel` and removes the corresponding guide in angular.io. PR Close angular#37190

This commit removes `ng-add` and `ng-new` schematics for the Bazel Builder, and update the corresponding BUILD files. PR Close angular#37190

This commit removes "Building with Bazel" section from angular.io navigation list and Angular CLI landing page. PR Close angular#37190

This commit removes aio/content/guide/bazel.md from the Bazel list in pullapprove since Bazel builder has been deprecated and the doc has been deleted. PR Close angular#37190

This commit removes the integration test for schematics in `@angular/bazel` that is used to generate a Bazel builder. The Bazel builder has been deprecated. PR Close angular#37190

…ar#37190) This commit adds a 301 redirect for /guide/bazel on angular.io to the deprecation doc for Angular Bazel schematics in Angular repo. PR Close angular#37190

This commit adds an exception for "guide/bazel" to the navigationUrls in the Service Worker config. This is needed for redirection to work. PR Close angular#37190

…ular#37190) This commit removes the fields for ng-add, schematics and builder from package.json of `@angular/bazel`. PR Close angular#37190

This commit improves some wording in the deprecation doc for Bazel builder and schematics in `@angular/bazel` and fixes the formatting. PR Close angular#37190

…7190) This commit adds a link to the Bazel prototype for orchestrating multiple CLI architects and also adds a link to the #angular channel in the Bazel Slack workspace. PR Close angular#37190

…ngular#37190) This commit adds Bazel builder and schematics to the global list of deprecations in Angular. A link to the migration doc is added. PR Close angular#37190

angular-automatic-lock-bot · 2020-06-29T16:11:42Z

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.}