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

Update test command documentation #9

Merged
merged 1 commit into from
Jul 29, 2023

Conversation

stephanecopin
Copy link
Contributor

Updates the documentation accounting for the changes in tuist/tuist#5135.

@netlify
Copy link

netlify bot commented Jul 3, 2023

Deploy Preview for tuist-docs failed.

Name Link
🔨 Latest commit bea4ea2
🔍 Latest deploy log https://app.netlify.com/sites/tuist-docs/deploys/64a2dd45dc1399000821c6ca

@danieleformichelli
Copy link
Contributor

@pepicrft this is why I don't like having docs and tuist on separate repos.
The PR on tuist and this one should be an atomic change.

Is there any reason to prefer having this as a separate repo?

@pepicrft
Copy link
Contributor

pepicrft commented Jul 3, 2023

Is there any reason to prefer having this as a separate repo?

Yes, first and most importantly, I'm working on a new documentation that builds upon Swift Catalogs and lives in the tuist/tuist repository and the experience is tightly integrated into Xcode. So this is only a temporary move until I have all the documentation ready.

Apart from that, the amount of complexity we've inherited in the repository outweighs the value we get out of being able to ship atomic PRs. Although it's hard to see it once you are familiar with the repository structure, for people coming across the repository for the first time is somewhat overwhelming. I captured part of it in this issue.

@pepicrft
Copy link
Contributor

@all-contributors add @stephanecopin for documentation

@allcontributors
Copy link
Contributor

@pepicrft

I've put up a pull request to add @stephanecopin! 🎉

@pepicrft pepicrft merged commit eafa050 into tuist:main Jul 29, 2023
0 of 5 checks passed
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