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

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
@vercel
Copy link
Contributor

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

@vercel
Copy link
Contributor

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
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants