Skip to content

[various] Run swift-format on example app Swift files - #5931

Merged
auto-submit[bot] merged 2 commits into
flutter:mainfrom
jmagman:swift-format-examples
Jan 19, 2024
Merged

[various] Run swift-format on example app Swift files#5931
auto-submit[bot] merged 2 commits into
flutter:mainfrom
jmagman:swift-format-examples

Conversation

@jmagman

Copy link
Copy Markdown
Member

Run swift-format linter/formatter on Swift files in example apps. This doesn't touch any Swift files in non-example code.

Part of turning on swift-format CI formatting #5928

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 relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/packages repo does use dart format.)
  • I signed the CLA.
  • The title of the PR starts with the name of the package surrounded by square brackets, e.g. [shared_preferences]
  • I listed at least one issue that this PR fixes in the description above.
  • I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
  • I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
  • 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.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@stuartmorgan-gstuartmorgan-g left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

Comment threadpackages/animations/CHANGELOG.md Outdated
@@ -1,3 +1,7 @@
## NEXT

* Fixes Swift format issues in example app.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Optional nit: you could revert all of the CHANGELOG changes, as this would fall under the "only affect[s] development of the package" exemption; formatting of the boilerplate runner code isn't going to be of interest to clients.

@jmagmanjmagman added override: no versioning needed Override the check requiring version bumps for most changes override: no changelog needed Override the check requiring CHANGELOG updates for most changes autosubmit Merge PR when tree becomes green via auto submit App labels Jan 19, 2024
@auto-submit
auto-submitBot merged commit 4dd91ce into flutter:mainJan 19, 2024
@jmagman
jmagman deleted the swift-format-examples branch January 19, 2024 04:55
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 19, 2024
auto-submitBot pushed a commit to flutter/flutter that referenced this pull request Jan 19, 2024
flutter/packages@83c2c4d...129e08c
2024-01-19 magder@google.com [pigeon] Run swift-format on ungenerated example app Swift files (flutter/packages#5934)
2024-01-19 magder@google.com [path_provider_foundation] Run swift-format on Swift files (flutter/packages#5935)
2024-01-19 magder@google.com [shared_preferences_foundation] Run swift-format on Swift files (flutter/packages#5933)
2024-01-19 magder@google.com [various] Run swift-format on example app Swift files (flutter/packages#5931)
2024-01-19 magder@google.com [camera_avfoundation] Remove development team from Xcode example app (flutter/packages#5930)
2024-01-18 jsharp83@gmail.com [webview_flutter_wkwebview] Add javascript panel interface for wkwebview (flutter/packages#5795)
2024-01-18 amirpanahandeh@yahoo.com [two_dimensional_scrollables] Fix must_call_super (flutter/packages#5921)
2024-01-18 cedvandenbosch@gmail.com [pointer_interceptor] fix width and height unset warning on web platform (flutter/packages#5864)
2024-01-18 49699333+dependabot[bot]@users.noreply.github.com Bump github/codeql-action from 3.23.0 to 3.23.1 (flutter/packages#5922)
2024-01-18 stuartmorgan@google.com [google_maps_flutter] Clean up iOS example project (flutter/packages#5925)
2024-01-18 magder@google.com [ci] Add flags to formatter command to decide which formatters to run (flutter/packages#5905)
2024-01-18 15619084+vashworth@users.noreply.github.com Update tests to Xcode 15 and iOS 17 simulator (flutter/packages#5914)
2024-01-18 engine-flutter-autoroll@skia.org Roll Flutter from def6af0 to f77f824 (25 revisions) (flutter/packages#5924)
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-flutter-autoroll
Please CC flutter-ecosystem@google.com,rmistry@google.com 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
creatorpiyush pushed a commit to creatorpiyush/packages that referenced this pull request Jun 10, 2026
Run `swift-format` linter/formatter on Swift files in example apps. This doesn't touch any Swift files in non-example code.
Part of turning on swift-format CI formatting flutter#5928
bisor0627 pushed a commit to bisor0627/packages that referenced this pull request Jun 19, 2026
Run `swift-format` linter/formatter on Swift files in example apps. This doesn't touch any Swift files in non-example code.
Part of turning on swift-format CI formatting flutter#5928
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmitMerge PR when tree becomes green via auto submit Appoverride: no changelog neededOverride the check requiring CHANGELOG updates for most changesoverride: no versioning neededOverride the check requiring version bumps for most changesp: animationsp: dynamic_layoutsThe dynamic_layouts packagep: file_selectorp: flutter_adaptive_scaffoldp: flutter_imagep: flutter_markdownp: go_routerp: palette_generatorp: path_providerp: pointer_interceptorp: quick_actionsp: rfwRemote Flutter Widgetsp: shared_preferencesplatform-iosplatform-macos

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@jmagman@stuartmorgan-g@hellohuanlin