Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@jason-simmons
Copy link
Member

These tests are sometimes hanging when the flutter_tester deletes the ContextVK and ConcurrentMessageLoop during shutdown.

…abled

These tests are sometimes hanging when the flutter_tester deletes the
ContextVK and ConcurrentMessageLoop during shutdown.
Copy link
Contributor

@jonahwilliams jonahwilliams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jason-simmons jason-simmons added autosubmit Merge PR when tree becomes green via auto submit App warning: land on red to fix tree breakage Override tree-status signal (land even with closed tree), combine with the autosubmit label. labels Oct 12, 2023
@flutter-dashboard
Copy link

Golden file changes have been found for this pull request. Click here to view and triage (e.g. because this is an intentional change).

If you are still iterating on this change and are not ready to resolve the images on the Flutter Gold dashboard, consider marking this PR as a draft pull request above. You will still be able to view image results on the dashboard, commenting will be silenced, and the check will not try to resolve itself until marked ready for review.

Changes reported for pull request #46854 at sha 2222c87

@auto-submit auto-submit bot merged commit bfa82c1 into flutter:main Oct 12, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 13, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Oct 13, 2023
…136506)

flutter/engine@dee90f1...ffb3b5b

2023-10-13 [email protected] Roll Dart SDK from 34c06f0bcb29 to b0d65ad14354 (3 revisions) (flutter/engine#46859)
2023-10-12 [email protected] Temporarily remove the runs of the engine Dart tests with Impeller enabled (flutter/engine#46854)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
CaseyHillers pushed a commit to CaseyHillers/engine that referenced this pull request Oct 19, 2023
…abled (flutter#46854)

These tests are sometimes hanging when the flutter_tester deletes the ContextVK and ConcurrentMessageLoop during shutdown.
CaseyHillers pushed a commit to CaseyHillers/engine that referenced this pull request Oct 19, 2023
…abled (flutter#46854)

These tests are sometimes hanging when the flutter_tester deletes the ContextVK and ConcurrentMessageLoop during shutdown.
CaseyHillers pushed a commit that referenced this pull request Oct 19, 2023
…er en… (#47112)

…abled (#46854)

These tests are sometimes hanging when the flutter_tester deletes the
ContextVK and ConcurrentMessageLoop during shutdown.

Co-authored-by: Jason Simmons <[email protected]>
harryterkelsen pushed a commit that referenced this pull request Oct 23, 2023
…abled (#46854)

These tests are sometimes hanging when the flutter_tester deletes the ContextVK and ConcurrentMessageLoop during shutdown.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

autosubmit Merge PR when tree becomes green via auto submit App warning: land on red to fix tree breakage Override tree-status signal (land even with closed tree), combine with the autosubmit label. will affect goldens

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants