Skip to content

Conversation

@andrewkolos
Copy link
Contributor

@andrewkolos andrewkolos commented Feb 7, 2023

Cherry-pick for #120167. Fixes: #120208.

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • 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.
  • All existing and new tests are passing.

@flutter-dashboard flutter-dashboard bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Feb 7, 2023

testWithoutContext('wildcard directories do not include subdirectories', () async {
final Platform platform = FakePlatform();
final FileSystem fs = MemoryFileSystem.test();
Copy link
Contributor

Choose a reason for hiding this comment

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

This test failed on ToT and had to be reverted. To make this more stable, you'll need to change this to use testUsingContext and getFlutterRoot()

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.

Test needs to be fixed

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

@godofredoc godofredoc merged commit 4c37964 into flutter:flutter-3.7-candidate.1 Feb 8, 2023
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Feb 9, 2023
…#3185)

* 9bbe247fc Fix lexer issue where select/plural/other/underscores cannot be in identifier names. (#119190) (flutter/flutter#119862)

* 4c3796466 [cp] Stop recursively including assets from asset folders (flutter/flutter#120209)

* 994429713 Roll engine version. (flutter/flutter#120324)
@andrewkolos andrewkolos deleted the cp-stop-recursive-asset-search-2 branch April 21, 2023 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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