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

docs: fixing --no-cache alert #10653

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 3 commits into from
Jul 10, 2025
Merged

docs: fixing --no-cache alert #10653

merged 3 commits into from
Jul 10, 2025

Conversation

0ri0nexe
Copy link
Contributor

@0ri0nexe 0ri0nexe commented Jul 9, 2025

Description

Minimal contribution because the alert of --no-cache deprecation gave a wrong command to use now with newer versions

Testing Instructions

In a project with a recent enough version of turbo such as 2.5.4 :
turbo run build --no-cache

--> You'll receive the alert on the use of --no-cache

@0ri0nexe 0ri0nexe requested a review from a team as a code owner July 9, 2025 14:42
@turbo-orchestrator turbo-orchestrator bot added the needs: triage New issues get this label. Remove it after triage label Jul 9, 2025
Copy link

vercel bot commented Jul 9, 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 Jul 10, 2025 4:49pm
examples-designsystem-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 10, 2025 4:49pm
examples-native-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 10, 2025 4:49pm
examples-tailwind-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 10, 2025 4:49pm
examples-vite-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 10, 2025 4:49pm
turbo-site ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 10, 2025 4:49pm

Copy link

vercel bot commented Jul 9, 2025

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

A member of the Team first needs to authorize it.

@0ri0nexe 0ri0nexe changed the title chore:fixing --no-cache alert docs:fixing --no-cache alert Jul 9, 2025
@0ri0nexe 0ri0nexe changed the title docs:fixing --no-cache alert docs: fixing --no-cache alert Jul 9, 2025
@0ri0nexe 0ri0nexe requested a review from anthonyshew as a code owner July 9, 2025 15:16
@turbo-orchestrator turbo-orchestrator bot added the area: docs Improvements or additions to documentation label Jul 9, 2025
@@ -399,7 +399,7 @@ impl Args {
if run_args.no_cache {
warn!(
"--no-cache is deprecated and will be removed in a future major version. Use \
--cache=local:r,remote:r"
Copy link
Contributor

Choose a reason for hiding this comment

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

This is the correct invocation to recommend. --no-cache would prevent writing, not reading.

This confusion is part of the reason we changed the API.

@anthonyshew
Copy link
Contributor

Hey, @0ri0nexe, thanks for this! We'll need to change the recommendation in the CLI back to how it was, but happy to merge in the docs update.

unfixed --no-cache comment
@0ri0nexe
Copy link
Contributor Author

My bad i really thought --no-cache didn't make any cache reading

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.

No worries, thank you!

@anthonyshew anthonyshew enabled auto-merge (squash) July 10, 2025 16:47
@anthonyshew anthonyshew merged commit 2ffe101 into vercel:main Jul 10, 2025
30 of 35 checks passed
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 needs: triage New issues get this label. Remove it after triage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants