这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@JamesHenry
Copy link
Contributor

The deleted statement was never accurate in Nx. We briefly openly explored some alternative solutions and pricing structures for self-hosted caching but this never landed as a requirement in Nx. We took on the feedback loud and clear that our initial thinking was misguided and we changed course before it landed.

We offer a broader array of free cache hosting options than Turborepo, with the OpenAPI spec being the exact equivalent of how Turborepo chooses to approach the problem, you can learn more here:

https://nx.dev/recipes/running-tasks/self-hosted-caching

For this change, I went for the most generous correction I could, but you may decide the overall section is redundant without this key incorrect point.

While I'm here, I'd suggest you run through this guide yourself again because it doesn't really make sense currently. If you run the command you have listed Nx generates package manager workspaces, no project.json files, and package.json files for each project, but then you tell people to add these things because that's "only how Turborepo does things" (not quoting but that's the inaccurate gist)...

Finally, as pointed out last time your nx.json to turbo.json comparison is very, very far from apples to apples. On the nx side you are demonstrating the configuration of a bunch of capabilities that Turborepo simply does not have and then saying "see, look, nx requires more config", which doesn't seem fair 🤷

Nx can work exactly how Turborepo works, and it can also work with optional plugins, users have complete freedom of choice. If you'd like to collaborate on further rewrites I'm totally open to that, but I thought I would keep this PR very focused on this critical correction for now.

@JamesHenry JamesHenry requested review from a team and anthonyshew as code owners May 8, 2025 20:16
@JamesHenry JamesHenry requested a review from chris-olszewski May 8, 2025 20:16
@turbo-orchestrator turbo-orchestrator bot added area: docs Improvements or additions to documentation needs: triage labels May 8, 2025
@JamesHenry JamesHenry changed the title chore(repo): [correction] nx remote caching is free like tuborepo chore(repo): [correction] nx remote caching is free like turborepo May 8, 2025
@vercel
Copy link
Contributor

vercel bot commented May 8, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
examples-basic-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 8, 2025 8:52pm
examples-designsystem-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 8, 2025 8:52pm
examples-gatsby-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 8, 2025 8:52pm
examples-native-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 8, 2025 8:52pm
examples-tailwind-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 8, 2025 8:52pm
examples-vite-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 8, 2025 8:52pm
turbo-site ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 8, 2025 8:52pm

@vercel
Copy link
Contributor

vercel bot commented May 8, 2025

@JamesHenry is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor

@anthonyshew anthonyshew left a comment

Choose a reason for hiding this comment

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

Glad to hear of the change of course! In that case, let's delete the section entirely since its no longer a point of difference. 👍

@JamesHenry JamesHenry requested a review from anthonyshew May 8, 2025 20:28
@anthonyshew anthonyshew changed the title chore(repo): [correction] nx remote caching is free like turborepo docs: [correction] nx remote caching is free like turborepo May 8, 2025
@anthonyshew anthonyshew changed the title docs: [correction] nx remote caching is free like turborepo docs: [correction] nx remote caching is free like Turborepo May 8, 2025
@anthonyshew anthonyshew enabled auto-merge (squash) May 8, 2025 20:37
Copy link
Contributor

@anthonyshew anthonyshew left a comment

Choose a reason for hiding this comment

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

Thanks!

@anthonyshew anthonyshew merged commit 13f3210 into vercel:main May 8, 2025
32 of 36 checks passed
@JamesHenry
Copy link
Contributor Author

Thank you!

@JamesHenry JamesHenry deleted the nx-corrections branch May 8, 2025 20:59
@anthonyshew anthonyshew changed the title docs: [correction] nx remote caching is free like Turborepo docs: nx remote caching has changed to be free like Turborepo May 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: docs Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants