Skip to content

[cp] Stop recursively including assets from asset folders - #120209

Merged
godofredoc merged 3 commits into
flutter:flutter-3.7-candidate.1from
andrewkolos:cp-stop-recursive-asset-search-2
Feb 8, 2023
Merged

godofredoc merged 3 commits into
flutter:flutter-3.7-candidate.1from
andrewkolos:cp-stop-recursive-asset-search-2

Conversation

@andrewkolos

@andrewkolos andrewkolos commented Feb 7, 2023 •

Copy link
Copy Markdown
Contributor

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
Copy Markdown
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()

@jonahwilliams jonahwilliams 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.

Test needs to be fixed

@jonahwilliams jonahwilliams 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.

LGTM

@godofredoc
godofredoc merged commit 4c37964 into flutter:flutter-3.7-candidate.1 Feb 8, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 9, 2023
auto-submit Bot pushed a commit to flutter/packages that referenced this pull request Feb 9, 2023
* 9bbe247 Fix lexer issue where select/plural/other/underscores cannot be in identifier names. (#119190) (flutter/flutter#119862)

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

* 9944297 Roll engine version. (flutter/flutter#120324)
@andrewkolos
andrewkolos deleted the cp-stop-recursive-asset-search-2 branch April 21, 2023 19:53
creatorpiyush pushed a commit to creatorpiyush/packages that referenced this pull request Jun 10, 2026
…3185)

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

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

* 9944297 Roll engine version. (flutter/flutter#120324)
bisor0627 pushed a commit to bisor0627/packages that referenced this pull request Jun 19, 2026
…3185)

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

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

* 9944297 Roll engine version. (flutter/flutter#120324)
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