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

[LAUNCH DOCS] Add instructions what needs to be installed, and how to README #757

Closed
mikekgfb opened this issue May 12, 2024 · 1 comment
Assignees

Comments

@mikekgfb
Copy link
Contributor

mikekgfb commented May 12, 2024

At present, running the instructions in the README will fail for the xcode project. See #755

At a minimum we should specify what should be installed and what the minimum xcode version (and any other requirements) are?

Also, I would expect this to fail even then, because like this might be GUI based with no fully scriptable set of instructions (plus it's not clear we'd want the script instructions when most devs are more likely going to like to start around with the GUI builder?). So, how can/should we test iOS app build in open source?

As a corollary, how do we automate testing of README for correctness? (and maybe the answer is "it's too involved", and that's OK if that turns out to be the right answer)

cc: @byjlw @shoumikhin

@mikekgfb mikekgfb changed the title [LAUNCH BLOCKER] Add instructions what needs to be installed, and how to README [LAUNCH DOCS] Add instructions what needs to be installed, and how to README May 12, 2024
@shoumikhin
Copy link
Contributor

@mikekgfb is this issue to flag that the current readme instructions don't work, or that they need to be expanded with more details, or if we can automate those instructions and run on CI?

shoumikhin added a commit that referenced this issue Jun 4, 2024
shoumikhin added a commit that referenced this issue Jun 4, 2024
* Update README.md

Addressing #757

* Update README.md
malfet pushed a commit that referenced this issue Jul 17, 2024
* Update README.md

Addressing #757

* Update README.md
malfet pushed a commit that referenced this issue Jul 17, 2024
* Update README.md

Addressing #757

* Update README.md
malfet pushed a commit that referenced this issue Jul 17, 2024
* Update README.md

Addressing #757

* Update README.md
malfet pushed a commit that referenced this issue Jul 17, 2024
* Update README.md

Addressing #757

* Update README.md
malfet pushed a commit that referenced this issue Jul 17, 2024
* Update README.md

Addressing #757

* Update README.md
malfet pushed a commit that referenced this issue Jul 17, 2024
* Update README.md

Addressing #757

* Update README.md
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

No branches or pull requests

3 participants