Sitelet https://github.com/flutter/flutter/pull/190629
Skip to content

Revert: Enable Gradle cache for a single test target - #190629

Merged
jtmcdole merged 1 commit into
flutter:masterfrom
flutteractionsbot:revert-190474-1785961985
Aug 5, 2026
Merged

Revert: Enable Gradle cache for a single test target#190629
jtmcdole merged 1 commit into
flutter:masterfrom
flutteractionsbot:revert-190474-1785961985

Conversation

@flutteractionsbot

@flutteractionsbot flutteractionsbot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Reverts: Enable Gradle cache for a single test target

Initiated by: @mboetger

Reason for reverting: broke the tree with spaces.

Original PR Author: @mboetger

Reviewed By: @jtmcdole

The original PR description is provided below:

Enables Gradle CI Cache for single test target.

Pre-launch Checklist

  • I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
  • I read the [AI contribution guidelines] and understand my responsibilities, or I am not using AI tools.
  • I read the [Tree Hygiene] wiki page, which explains my responsibilities.
  • I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
  • I signed the [CLA].
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is [test-exempt].
  • I followed the [breaking change policy] and added [Data Driven Fixes] where supported.
  • All existing and new tests are passing.

@flutteractionsbot flutteractionsbot mentioned this pull request Aug 5, 2026
Merged
10 tasks done
@flutter-dashboard flutter-dashboard Bot added the CICD Run CI/CD label Aug 5, 2026
gemini-code-assist[bot]
gemini-code-assist Bot reviewed Aug 5, 2026
View reviewed changes

@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 updates the .ci.yaml configuration file by removing the gradle_dists dependency from the tool_tests_commands shard targets. There are no review comments, and I have no feedback to provide.

@jtmcdole jtmcdole added the emergency Jump the queue; land PR in front of all others; only use for emergencies label Aug 5, 2026
@flutter-dashboard

flutter-dashboard Bot commented Aug 5, 2026

Copy link
Copy Markdown

Detected the emergency label.

If you add the autosubmit label, the bot will wait until all presubmits pass but ignore the tree status, allowing fixes for tree breakages while still validating that they don't break any existing presubmits.

The "Merge" button is also unlocked. To bypass presubmits as well as the tree status, press the GitHub "Add to Merge Queue".

@jtmcdole
jtmcdole added this pull request to the merge queue Aug 5, 2026
Merged via the queue into flutter:master with commit 07ab4d3 Aug 5, 2026
26 of 27 checks passed
@flutteractionsbot
flutteractionsbot deleted the revert-190474-1785961985 branch August 5, 2026 22:02
This was referenced Aug 6, 2026
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jtmcdole jtmcdole jtmcdole approved these changes
+1 more reviewer
@gemini-code-assist gemini-code-assist[bot] gemini-code-assist[bot] left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

CICD Run CI/CD emergency Jump the queue; land PR in front of all others; only use for emergencies

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@flutteractionsbot @jtmcdole