Skip to content

feat: experimental tool tests workflow (attempt 2) #169768

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

Merged
merged 2 commits into from
May 30, 2025

Conversation

jtmcdole
Copy link
Member

@jtmcdole jtmcdole commented May 30, 2025

  • remote repo: set upstream and get tags (flutter tool reqs)
  • pull request to flutter: checkout the PR, not the merge commit
  • local runs: update utils.dart to safely call terminal width

Now act -W .github/workflows/tool-test-general.yml works locally on the command line.

Future improvements:

  • Offer another way to set the "version" rather than tags; this causes a longer checkout time.
  • Multi-proc the tests and use as many cores as possible?
  • Solve for packages downloading?

3.5 minutes isn't terrible.

@jtmcdole jtmcdole requested a review from matanlurey as a code owner May 30, 2025 16:59
@jtmcdole jtmcdole added the autosubmit Merge PR when tree becomes green via auto submit App label May 30, 2025
@jtmcdole jtmcdole requested a review from zanderso May 30, 2025 17:06
@jtmcdole jtmcdole changed the title fix: update experiment to work in different conditions fix: update tool workflow experiment for different conditions May 30, 2025
@zanderso
Copy link
Member

@jason-simmons is reverting #169706 as it's blocking the Dart rolls. @jtmcdole you are OOO today, so let's go with the revert and pick this up again on Monday =)

@jtmcdole
Copy link
Member Author

jtmcdole commented May 30, 2025

@jason-simmons is reverting #169706 as it's blocking the Dart rolls. @jtmcdole you are OOO today, so let's go with the revert and pick this up again on Monday =)

Ah, this would have fixed it, but makes sense. I can just rebase after the revert and it should work on Monday

Copy link
Contributor

auto-submit bot commented May 30, 2025

autosubmit label was removed for flutter/flutter/169768, because Pull request flutter/flutter/169768 is not in a mergeable state.

@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label May 30, 2025
jtmcdole added 2 commits May 30, 2025 13:53
 - remote repo: set upstream and get tags (flutter tool reqs)
 - pull request to flutter: checkout the PR, not the merge commit
 - local runs: update utils.dart to safely call terminal width

Now `act -W .github/workflows/tool-test-general.yml` works locally on
the command line.
@jtmcdole jtmcdole force-pushed the updateToolsTestWorkflow branch from 6d772ac to fd87ec7 Compare May 30, 2025 20:54
@jtmcdole
Copy link
Member Author

@zanderso - rebased after the revert - you can see the tests passes (hint to future self: always add the workflow to the paths: to see it working)

@jtmcdole jtmcdole changed the title fix: update tool workflow experiment for different conditions feat: experimental tool tests workflow (attempt 2) May 30, 2025
@jtmcdole jtmcdole added the autosubmit Merge PR when tree becomes green via auto submit App label May 30, 2025
@auto-submit auto-submit bot added this pull request to the merge queue May 30, 2025
Merged via the queue into flutter:master with commit 1946081 May 30, 2025
150 of 151 checks passed
@flutter-dashboard flutter-dashboard bot removed the autosubmit Merge PR when tree becomes green via auto submit App label May 30, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 31, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 31, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 1, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 1, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 2, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 2, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 2, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 2, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 2, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jun 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants