-
Notifications
You must be signed in to change notification settings - Fork 96
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix postsubmit rerun based on checkrun #3675
base: main
Are you sure you want to change the base?
Conversation
04e9ded
to
67adf4b
Compare
Just a heads up that this will conflict with the buildbucket changes. If we can, can we wait for those changes to land? I can help convert this afterwards if you like. SG. /cc @stuartmorgan I will wait for the Buildbucket V2 changes to land before merging this. |
} | ||
tags['current_attempt'] = <String>[(taskDocument.attempts!).toString()]; | ||
log.info('Updated input ${build.input!} tags $tags'); | ||
final BatchRequest request = BatchRequest( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess I am not understanding, is this really a postsubmit task that is run in presubmit for packages?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Validated this works with https://github.com/flutter/packages/runs/24162936475. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Fixes: flutter/flutter#147007