Skip to content

Conversation

@goderbauer
Copy link
Member

Reverts #121215

This commit was breaking the ability to run tests straight from IntelliJ: When right-clicking a test and selecting Run foo it just spins indefinitely on the "Instantiating tests..." step. When I backed out this change locally, the problem went away.

@flutter-dashboard flutter-dashboard bot added framework flutter/packages/flutter repository. See also f: labels. platform-ios iOS applications specifically c: contributor-productivity Team-specific productivity, code health, technical debt. tool Affects the "flutter" command-line tool. See also t: labels. labels Feb 27, 2023
@goderbauer goderbauer added the autosubmit Merge PR when tree becomes green via auto submit App label Feb 27, 2023
@auto-submit auto-submit bot merged commit 275ab9c into master Feb 27, 2023
@auto-submit auto-submit bot deleted the revert-121215-reland_remove_observatory_references branch February 27, 2023 23:46
@XilaiZhang
Copy link
Contributor

Thanks for the revert! @goderbauer can you also cc me next time :) Would love to include this change in the roll earlier

@goderbauer
Copy link
Member Author

@XilaiZhang Sure, what kind of changes are you interested in? Just any kind of rollback?

@XilaiZhang
Copy link
Contributor

for sure, rollbacks are very helpful! It would be easier to include in the roll than cherry picking the roll back afterwards

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App c: contributor-productivity Team-specific productivity, code health, technical debt. framework flutter/packages/flutter repository. See also f: labels. platform-ios iOS applications specifically tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants