Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

chore(local): use //cmd/frontend:frontend_nobundle for bazel commandsets#64085

Merged
Strum355 merged 1 commit into
mainfrom
nsc/sg-bazelcommand-frontend
Jul 25, 2024
Merged

chore(local): use //cmd/frontend:frontend_nobundle for bazel commandsets#64085
Strum355 merged 1 commit into
mainfrom
nsc/sg-bazelcommand-frontend

Conversation

@Strum355

@Strum355 Strum355 commented Jul 25, 2024

Copy link
Copy Markdown
Contributor

Changes in client-side code should not result in expensive rebuilds of frontend locally when using bazel with sg start. For some reason, iBazel still thinks it needs to be rebuilt (possibly it uses query instead of cquery and therefore doesnt resolve selects), but bazel underneath determines nothing needs to be rebuilt so theres very little cost

Test plan

Command set with just frontend in a bazelCommands section and then sg start <command set>

Changelog

@Strum355 Strum355 requested a review from a team July 25, 2024 16:45
@cla-bot cla-bot Bot added the cla-signed label Jul 25, 2024
@Strum355 Strum355 enabled auto-merge (squash) July 25, 2024 16:49
@Strum355 Strum355 merged commit 5507a4a into main Jul 25, 2024
@Strum355 Strum355 deleted the nsc/sg-bazelcommand-frontend branch July 25, 2024 16:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants