Skip to content
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

feat: quickstart in swift is now runnable #78

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

riverKanies
Copy link
Contributor

If we do want to move forward with running swift examples as scripts like this, here's the updated quickstart

Matthew seems fine with it

@reez here's the updated version, only required minor changes from the basics.md you had

@reez
Copy link
Contributor

reez commented Oct 29, 2024

Code looks good, so whatever way you decide to do this (project, no project, etc) is good w me-

@riverKanies
Copy link
Contributor Author

ok cool, I'll play around with a minimal xcode proj and see if that would make more sense

@riverKanies riverKanies marked this pull request as draft October 29, 2024 19:46
@notmandatory
Copy link
Member

This looks good to me too, you can close #82. I thought you were using some sort of script tool but looks like you're just using swift package init --type executable which seems a minimal way to get started.

@riverKanies riverKanies marked this pull request as ready for review October 30, 2024 18:12
@riverKanies
Copy link
Contributor Author

@thunderbiscuit looks like this is the direction the team wants to go

The main open q left to fully address here is exactly how much xcode specific details to include in the instructions. I think we can outline that stuff and also mention that the actual code example is not an xcode project. That's what it currently does but we can iterate a little on that

@riverKanies riverKanies force-pushed the feature/swift-quickstart branch from 0ee7c5a to 9350e27 Compare October 30, 2024 18:15
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.

3 participants