Skip to content

experiment for updating dashboard checks status to in progress - #5128

Merged
auto-submit[bot] merged 4 commits into
flutter:mainfrom
ievdokdm:191005-re-request
Aug 18, 2026
Merged

auto-submit[bot] merged 4 commits into
flutter:mainfrom
ievdokdm:191005-re-request

Conversation

@ievdokdm

Copy link
Copy Markdown
Contributor

@flutter-dashboard flutter-dashboard Bot added the CICD Run CI/CD label Aug 18, 2026

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

Copy link
Copy Markdown

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 enhances the dashboard check run updates in LuciBuildService by adding logging, a detailed output (title and summary), and a detailsUrl linking to the Flutter dashboard. The corresponding tests have been updated to match these new parameters. The review feedback notes that dashboardChecks.headSha is nullable and could result in a broken URL containing sha=null, and suggests using the non-nullable local variable commitSha instead.

ievdokdm and others added 2 commits August 17, 2026 17:51
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@ievdokdm
ievdokdm requested a review from jtmcdole August 18, 2026 16:32
@ievdokdm ievdokdm added the autosubmit Merge PR when tree becomes green via auto submit App. label Aug 18, 2026
@auto-submit
auto-submit Bot merged commit 7af83c4 into flutter:main Aug 18, 2026
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App. CICD Run CI/CD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants