Skip to content

build: remove BES configuration from remote config#3682

Merged
alan-agius4 merged 1 commit into
angular:mainfrom
josephperrott:remove-bes
May 19, 2026
Merged

build: remove BES configuration from remote config#3682
alan-agius4 merged 1 commit into
angular:mainfrom
josephperrott:remove-bes

Conversation

@josephperrott
Copy link
Copy Markdown
Member

No description provided.

@josephperrott josephperrott added the action: merge The PR is ready for merge by the caretaker label May 18, 2026
@josephperrott josephperrott requested a review from alan-agius4 May 18, 2026 16:16
@angular-robot angular-robot Bot added the area: build & ci Related the build and CI infrastructure of the project label May 18, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request removes the Build Event Service (BES) configuration from the .bazelrc.common file. Feedback indicates that the cleanup is incomplete, as a BES-related instance name configuration remains on line 55 and should also be removed for consistency.

Comment thread .bazelrc.common
@alan-agius4 alan-agius4 merged commit 725f1f9 into angular:main May 19, 2026
12 checks passed
@alan-agius4
Copy link
Copy Markdown
Contributor

This PR was merged into the repository. The changes were merged into the following branches:

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants