Skip to content

Improve iOS readme #5990

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 8, 2024
Merged

Conversation

pytorchbot
Copy link
Collaborator

Summary:
Addressing feedback from D63264488

  • Rewrite the XCode setup flow for better clarity.
  • Added section numbering and reorder some of the flow, emphasizing which steps are recommended and which are optional (For example, user can manually build the package, but it's gonna be more involved and has to deal with lib linkage, etc)
  • Mentioned that package dependencies has been pre-configured and clarified instructions on how to change that if needed
  • Added more details on which package should link against which app target
  • Added more details on how to confirm which packages have been linked in case the options are greyed out

Differential Revision: D63403105

Summary:
Pull Request resolved: #5877

Addressing feedback from D63264488

- Rewrite the XCode setup flow for better clarity.
- Added section numbering and reorder some of the flow, emphasizing which steps are recommended and which are optional (For example, user can manually build the package, but it's gonna be more involved and has to deal with lib linkage, etc)
- Mentioned that package dependencies has been pre-configured and clarified instructions on how to change that if needed
- Added more details on which package should link against which app target
- Added more details on how to confirm which packages have been linked in case the options are greyed out

Reviewed By: shoumikhin

Differential Revision: D63403105

fbshipit-source-id: c9867efa2d48e3e37f39b7a98b9c644390bcd3b2
(cherry picked from commit f3de2bb)
This was referenced Oct 8, 2024
Copy link

pytorch-bot bot commented Oct 8, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/5990

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit 665d8ff with merge base eecf74f (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 8, 2024
@jackzhxng jackzhxng merged commit a9d7182 into release/0.4 Oct 8, 2024
39 checks passed
@jackzhxng jackzhxng deleted the cherry-pick-5877-by-pytorch_bot_bot_ branch October 8, 2024 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants