Skip to content

Goma is dead, long live RBE! - #195798

Merged
rmacnak-google merged 1 commit into
flutter:masterfrom
rmacnak-google:goma
Nov 4, 2026
Merged

rmacnak-google merged 1 commit into
flutter:masterfrom
rmacnak-google:goma

Conversation

@rmacnak-google

Copy link
Copy Markdown
Contributor

No description provided.

@rmacnak-google rmacnak-google added the CICD Run CI/CD label Nov 2, 2026
@github-actions github-actions Bot added the engine flutter/engine related. See also e: labels. label Nov 2, 2026

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

Copy link
Copy Markdown
Contributor

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 Goma-related arguments from various CI builder configuration files and the GN build helper script. Several JSON files, specifically linux_unopt.json and local_engine.json, contain syntax errors due to missing commas where the Goma arguments were removed, which will cause JSON parsing failures.

@rmacnak-google

Copy link
Copy Markdown
Contributor Author

@ievdokdm ievdokdm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@rmacnak-google
rmacnak-google added this pull request to the merge queue Nov 4, 2026
Merged via the queue into flutter:master with commit b9944cf Nov 4, 2026
18 checks passed
@rmacnak-google
rmacnak-google deleted the goma branch November 4, 2026 19:28
@jtmcdole jtmcdole added the revert Autorevert PR (with "Reason for revert:" comment) label Nov 4, 2026
@flutteractionsbot

Copy link
Copy Markdown
Contributor

Successfully created revert PR: #196001

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CICD Run CI/CD engine flutter/engine related. See also e: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants