Skip to content

Conversation

@LouiseHsu
Copy link
Contributor

Update to #156733

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].
  • I followed the [breaking change policy] and added [Data Driven Fixes] where supported.
  • All existing and new tests are passing.

@github-actions github-actions bot added tool Affects the "flutter" command-line tool. See also t: labels. a: desktop Running on desktop labels Nov 5, 2024
@LouiseHsu LouiseHsu requested a review from loic-sharma November 5, 2024 21:34
'To fix your Xcode project, '
'see https://github.com/flutter/flutter/issues/156733#issuecomment-2415359014 '
'for a workaround.',
'Error: Your Cocoapods may be out-of-date and unable to support synchronized groups/folders. '
Copy link
Member

@loic-sharma loic-sharma Nov 5, 2024

Choose a reason for hiding this comment

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

Nit: Google technical writing style prefers "might" over "may" unless it's a legal context

Suggested change
'Error: Your Cocoapods may be out-of-date and unable to support synchronized groups/folders. '
'Error: Your Cocoapods might be out-of-date and unable to support synchronized groups/folders. '

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i want a writing style formatter

'To fix your Xcode project, '
'see https://github.com/flutter/flutter/issues/156733#issuecomment-2415359014 '
'for a workaround.',
'Error: Your Cocoapods may be out-of-date and unable to support synchronized groups/folders. '
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
'Error: Your Cocoapods may be out-of-date and unable to support synchronized groups/folders. '
'Error: Your Cocoapods might be out-of-date and unable to support synchronized groups/folders. '

@loic-sharma
Copy link
Member

loic-sharma commented Nov 5, 2024

Could we also bump the recommended CocoaPods version:

const Version cocoaPodsMinimumVersion = Version.withText(1, 10, 0, '1.10.0');
const Version cocoaPodsRecommendedVersion = Version.withText(1, 13, 0, '1.13.0');

This will make flutter doctor recommend a CocoaPods upgrade.

Copy link
Member

@loic-sharma loic-sharma left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for updating this!

@LouiseHsu
Copy link
Contributor Author

Could we also bump the recommended CocoaPods version?

const Version cocoaPodsMinimumVersion = Version.withText(1, 10, 0, '1.10.0');
const Version cocoaPodsRecommendedVersion = Version.withText(1, 13, 0, '1.13.0');

🫡

@LouiseHsu LouiseHsu added the autosubmit Merge PR when tree becomes green via auto submit App label Nov 5, 2024
@auto-submit auto-submit bot merged commit dff1916 into flutter:master Nov 6, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Nov 6, 2024
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Nov 6, 2024
flutter/flutter@29d40f7...73546b3

2024-11-06 32538273+ValentinVignal@users.noreply.github.com Add test for `image.loading_builder.0.dart` (flutter/flutter#158248)
2024-11-06 kustermann@google.com Make flutter_tools use newest package:{native_assets_builder,native_assets_cli,native_toolchain_c} (flutter/flutter#158214)
2024-11-06 bruno.leroux@gmail.com Fix RawScrollbar examples and desktop test (flutter/flutter#158237)
2024-11-06 bruno.leroux@gmail.com Cleanup MenuAnchor and Improve DropdownMenu tests readability (flutter/flutter#158175)
2024-11-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from a3741d6248b7 to f03f11300a9d (2 revisions) (flutter/flutter#158222)
2024-11-06 louisehsu@google.com Update error message for Cocoapods support for synchronized groups/folders (flutter/flutter#158206)
2024-11-06 matanlurey@users.noreply.github.com Restore skipped iOS test by looping over `FakeAsync` elapse. (flutter/flutter#158204)
2024-11-06 41366638+kirya355@users.noreply.github.com fix: ensure draggable_scrollable_sheet respects shouldCloseOnMinExten� (flutter/flutter#156338)
2024-11-06 engine-flutter-autoroll@skia.org Roll Flutter Engine from e5e06c97c33c to a3741d6248b7 (14 revisions) (flutter/flutter#158218)
2024-11-06 matanlurey@users.noreply.github.com Forward fix `CupertinoDynamicColor` by adding `toARGB32()`. (flutter/flutter#158145)
2024-11-05 matanlurey@users.noreply.github.com Remove unused `enableObservatory` flag. (flutter/flutter#158202)
2024-11-05 matanlurey@users.noreply.github.com Remove observatory related TODO that is already fixed. (flutter/flutter#158205)
2024-11-05 nate.w5687@gmail.com Factor out "shaker" class (flutter/flutter#157748)
2024-11-05 fluttergithubbot@gmail.com Marks Mac_benchmark animated_complex_opacity_perf_macos__e2e_summary to be flaky (flutter/flutter#157424)
2024-11-05 andrewrkolos@gmail.com Increase subsharding for `Linux tool_integration_tests` (flutter/flutter#158196)
2024-11-05 32538273+ValentinVignal@users.noreply.github.com Add test for `raw_scrollbar.2.dart` (flutter/flutter#158161)
2024-11-05 andrewrkolos@gmail.com use root directory as the default for rootOverride in Cache.test constructor (flutter/flutter#158201)
2024-11-05 magder@google.com Kill interactive script job `xcdevice observe` processes on tool/daemon shutdown (flutter/flutter#157646)
2024-11-05 34465683+rkishan516@users.noreply.github.com Fix: Gap between prefix and suffix icon and input field in input deco� (flutter/flutter#152069)
2024-11-05 engine-flutter-autoroll@skia.org Roll Flutter Engine from f56401062e42 to e5e06c97c33c (1 revision) (flutter/flutter#158194)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC camillesimon@google.com,stuartmorgan@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Packages: 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
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 12, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 13, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 13, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 6, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: desktop Running on desktop autosubmit Merge PR when tree becomes green via auto submit App tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants