Skip to content

create Presubmit check-run in in-progress status - #5161

Open
ievdokdm wants to merge 1 commit into
flutter:mainfrom
ievdokdm:176973-in-progress
Open

ievdokdm wants to merge 1 commit into
flutter:mainfrom
ievdokdm:176973-in-progress

Conversation

@ievdokdm

Copy link
Copy Markdown
Contributor

For better usability create Presubmit check-run in In Progress status

@flutter-dashboard flutter-dashboard Bot added the CICD Run CI/CD label Sep 30, 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 introduces a status parameter to the GitHub check run utilities, defaulting to queued, and updates the LuciBuildService and Scheduler to set the status to inProgress when starting presubmit checks. It also updates the corresponding tests and mock files. The review feedback points out that manually editing the generated mock file mocks.mocks.dart is discouraged because those changes will be overwritten during the next code generation, and recommends regenerating the mocks using build_runner instead.

Comment thread packages/cocoon_integration_test/lib/src/utilities/mocks.mocks.dart
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CICD Run CI/CD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant