Skip to content

Conversation

@gaaclarke
Copy link
Member

Description of what this PR is changing or adding, and why:

Updated the swift example code to match the compiled swift example: https://github.com/flutter/flutter/blob/1b9502b007d01e1cc79dd86fc3300061522803a5/examples/platform_channel_swift/ios/Runner/AppDelegate.swift#L33

There were syntactic errors in the swift code which isn't checked like the dart code.

Issues fixed by this PR (if any):

PRs or commits this PR depends on (if any):

Presubmit checklist

  • This PR is marked as draft with an explanation if not meant to land until a future stable release.
  • This PR doesn’t contain automatically generated corrections (Grammarly or similar).
  • This PR follows the Google Developer Documentation Style Guidelines — for example, it doesn’t use i.e. or e.g., and it avoids I and we (first person).
  • This PR uses semantic line breaks of 80 characters or fewer.

@flutter-website-bot
Copy link
Collaborator

Visit the preview URL for this PR (updated for commit bc95120):

https://flutter-docs-prod--pr12110-updated-swift-example-uisce-nok55w1q.web.app

Copy link
Contributor

@sfshaza2 sfshaza2 left a comment

Choose a reason for hiding this comment

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

lgtm, once you and @stuartmorgan-g reach agreement

@gaaclarke gaaclarke merged commit 3190877 into flutter:main Jun 10, 2025
9 checks passed
ericwindmill pushed a commit that referenced this pull request Jun 24, 2025
_Description of what this PR is changing or adding, and why:_

Updated the swift example code to match the compiled swift example:
https://github.com/flutter/flutter/blob/1b9502b007d01e1cc79dd86fc3300061522803a5/examples/platform_channel_swift/ios/Runner/AppDelegate.swift#L33

There were syntactic errors in the swift code which isn't checked like
the dart code.

_Issues fixed by this PR (if any):_

_PRs or commits this PR depends on (if any):_

## Presubmit checklist

- [x] This PR is marked as draft with an explanation if not meant to
land until a future stable release.
- [x] This PR doesn’t contain automatically generated corrections
(Grammarly or similar).
- [x] This PR follows the [Google Developer Documentation Style
Guidelines](https://developers.google.com/style) — for example, it
doesn’t use _i.e._ or _e.g._, and it avoids _I_ and _we_ (first person).
- [x] This PR uses [semantic line
breaks](https://github.com/dart-lang/site-shared/blob/main/doc/writing-for-dart-and-flutter-websites.md#semantic-line-breaks)
of 80 characters or fewer.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants